RFR: 8367784: java/awt/Focus/InitialFocusTest/InitialFocusTest1.java failed with Wrong focus owner [v2]
Prasanta Sadhukhan
psadhukhan at openjdk.org
Thu Sep 18 04:35:33 UTC 2025
On Wed, 17 Sep 2025 17:36:09 GMT, Phil Race <prr at openjdk.org> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Use robot
>
> test/jdk/java/awt/Focus/InitialFocusTest/InitialFocusTest1.java line 60:
>
>> 58: app.setVisible(true);
>> 59: });
>> 60: Thread.sleep(1000);
>
> Why not use Robot so you can also use waitForIdle() ?
>
> And is there a particular reason you think not using the EDT is why the test failed in CI ?
Used robot now..Tried for another 100 iterations which is ok..
I guess the order of focusing would be more preserved if EDT is used to account for thread-safety
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27332#discussion_r2357483240
More information about the client-libs-dev
mailing list