On Wed, 20 Aug 2025 05:05:20 GMT, Anass Baya <ab...@openjdk.org> wrote:
>> **Analysis:** >> The getFocusableWindowState() method is not intended to modify the >> configuration; doing so can cause recursive re-entry on Linux. >> >> **Proposed Fix:** >> We are intentionally overriding getFocusableWindowState() to allow it to >> change the configuration, in order to verify that calling getScreenImOn() >> for ownerless windows does not throw any exceptions. >> To prevent recursive re-entry, we use a flag to ensure that >> getFocusableWindowState() is executed only once > > Anass Baya has updated the pull request incrementally with one additional > commit since the last revision: > > add space looks fine, please confirm that all tests are green. ------------- Marked as reviewed by serb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/26298#pullrequestreview-3153452042
