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