RFR: 8341000: Open source some of the AWT Window tests [v3]
Jayathirth D V
jdv at openjdk.org
Thu Oct 3 08:15:58 UTC 2024
On Thu, 3 Oct 2024 07:40:15 GMT, Abhishek Kumar <abhiscxk at openjdk.org> wrote:
>> Jayathirth D V has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update based on review comments
>
> test/jdk/java/awt/Window/InvalidFocusLostEventTest/InvalidFocusLostEventTest.java line 60:
>
>> 58: countDownLatch.await();
>> 59: if (fm.getFocusOwner() != b) {
>> 60: failed = true;
>
> I guess exception can be thrown here and then no need of `failed` variable and checking from finally block can be removed.
Good idea. Removed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21281#discussion_r1785812742
More information about the client-libs-dev
mailing list