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