RFR: 8341000: Open source some of the AWT Window tests [v4]

Jayathirth D V jdv at openjdk.org
Thu Oct 3 12:51:41 UTC 2024


On Thu, 3 Oct 2024 11:50:06 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Jayathirth D V has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove failed flag
>
> 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.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/21281#discussion_r1786167460


More information about the client-libs-dev mailing list