On Tue, 15 Jul 2025 19:52:15 GMT, Alisen Chung <[email protected]> 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

Reply via email to