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