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