RFR: 8361521 : BogusFocusableWindowState.java fails with StackOverflowError on Linux [v3]
Anass Baya
abaya at openjdk.org
Fri Aug 8 00:30:26 UTC 2025
On Thu, 31 Jul 2025 14:21:33 GMT, Anass Baya <abaya 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
>
> Anass Baya has updated the pull request incrementally with one additional commit since the last revision:
>
> Do not update Focusability when it is not needed
Converted to draft for more investigation. the proposed fix my cause issue when resizing
To avoid any issues when resizing the window, we only update focusability when it's necessary, not for every setVisible call
-------------
PR Comment: https://git.openjdk.org/jdk/pull/26298#issuecomment-3141351267
PR Comment: https://git.openjdk.org/jdk/pull/26298#issuecomment-3166209476
More information about the client-libs-dev
mailing list