On Thu, 3 Oct 2024 12:48:30 GMT, Jayathirth D V <[email protected]> wrote:
>> test/jdk/java/awt/Window/InvalidFocusLostEventTest/InvalidFocusLostEventTest.java >> line 98: >> >>> 96: // we should check focus after all events are processed, >>> 97: // since focus transfers are asynchronous >>> 98: EventQueue.invokeLater(() -> countDownLatch.countDown()); >> >> I guess we can just do countDown without it being in invokeLater... > > I have included countDown specifically inside asynchronous call of EDT > because of the comment mentioned above `we should check focus after all > events are processed, since focus transfers are asynchronous`. > > So that we actually check for focus after everything is processed in EDT. But I am not sure this is the correct way to ensure...We are trying to do away with invokeLater call from jtreg tests. I think it's more apt to call robot.waitForIdle to ensure all events are processed void [waitForIdle](https://docs.oracle.com/en/java/javase/22/docs/api/java.desktop/java/awt/Robot.html#waitForIdle())() Waits until all events currently on the event queue have been processed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21281#discussion_r1786173362
