RFR: 8368185: Test javax/swing/plaf/synth/SynthButtonUI/6276188/bug6276188.java failed: Synth ButtonUI does not handle PRESSED & MOUSE_OVER state [v2]

Prasanta Sadhukhan psadhukhan at openjdk.org
Sun Oct 5 05:52:45 UTC 2025


On Fri, 3 Oct 2025 14:30:29 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use latch sync
>
> test/jdk/javax/swing/plaf/synth/SynthButtonUI/6276188/bug6276188.java line 103:
> 
>> 101:             robot.waitForIdle();
>> 102:             latch.await();
>> 103:             robot.delay(1000);
> 
> Can this delay be reduced to 500 or less? Anyway, 1 second delay is better than 2 seconds.
> 
> `waitForIdle` should go after `await` to ensure all the pending events after `mousePressed` are processed.

Reducing delay further causes failure in CI so I will keep it

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27444#discussion_r2404299085


More information about the client-libs-dev mailing list