RFR: 8368185: Test javax/swing/plaf/synth/SynthButtonUI/6276188/bug6276188.java failed: Synth ButtonUI does not handle PRESSED & MOUSE_OVER state [v2]
Alexey Ivanov
aivanov at openjdk.org
Mon Oct 6 21:01:49 UTC 2025
On Sun, 5 Oct 2025 05:50:19 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> 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
I see.
I expected to have `waitForIdle` after `await` to ensure all other pending events including `paint` is handled.
robot.mousePress(InputEvent.BUTTON1_DOWN_MASK);
latch.await();
robot.waitForIdle();
robot.delay(1000);
In this case, it could be possible to reduce the delay… or maybe not.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27444#discussion_r2408545621
More information about the client-libs-dev
mailing list