On Wed, 17 Sep 2025 17:36:09 GMT, Phil Race <[email protected]> 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
