RFR: 8367784: java/awt/Focus/InitialFocusTest/InitialFocusTest1.java failed with Wrong focus owner [v2]
Damon Nguyen
dnguyen at openjdk.org
Thu Sep 18 16:20:34 UTC 2025
On Thu, 18 Sep 2025 04:32:40 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> Test was deproblemlisted in https://github.com/openjdk/jdk/pull/27136 without any modifications as it was passing in testing.
>> Since it again failed, added the execution in EDT to be more thread-safe.
>>
>> Fix is tested against several CI jobs each with several iterations link of which is in JBS
>
> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>
> Use robot
Confirmed that the test passes locally and on CI.
test/jdk/java/awt/Focus/InitialFocusTest/InitialFocusTest1.java line 75:
> 73: EventQueue.invokeAndWait(() -> {
> 74: app.dispose();
> 75: });
Suggestion:
EventQueue.invokeAndWait(() -> app.dispose());
Can use a lambda expression for the app disposal.
-------------
Marked as reviewed by dnguyen (Committer).
PR Review: https://git.openjdk.org/jdk/pull/27332#pullrequestreview-3240773963
PR Review Comment: https://git.openjdk.org/jdk/pull/27332#discussion_r2360135323
More information about the client-libs-dev
mailing list