On Thu, 18 Sep 2025 16:33:01 GMT, Prasanta Sadhukhan <[email protected]> 
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:
> 
>   lambda Fix

test/jdk/java/awt/Focus/InitialFocusTest/InitialFocusTest1.java line 69:

> 67:             // wait for the very very last focus event
> 68:             robot.delay(1000);
> 69:             if (app.button2 != focused) {

Since it is object comparison is it better to use` 
!app.button2.equals(focused)`?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/27332#discussion_r2360397841

Reply via email to