> **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:

  Revert changes to BogusFocusableWindowState.java from

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/26298/files
  - new: https://git.openjdk.org/jdk/pull/26298/files/9cfd22da..2019239f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=26298&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26298&range=00-01

  Stats: 12 lines in 1 file changed: 0 ins; 11 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/26298.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/26298/head:pull/26298

PR: https://git.openjdk.org/jdk/pull/26298

Reply via email to