RFR: 8361521 : BogusFocusableWindowState.java fails with StackOverflowError on Linux

[Anass Baya]

On Tue, 15 Jul 2025 19:52:15 GMT, Alisen Chung <achung at 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
>
>> The getFocusableWindowState() method is not intended to modify the configuration; doing so can cause recursive re-entry on Linux.
> 
> What's causing the recursive re-entry? Is the bug in the code path in linux when calling getFocusableWindowState?

Hello @alisenchung, @mrserb,
Do you agree with the new proposed fix ?
Thanks

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

PR Comment: https://git.openjdk.org/jdk/pull/26298#issuecomment-3197614367