On Thu, 31 Jul 2025 14:21:33 GMT, Anass Baya <ab...@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
