On Sun, 5 Oct 2025 05:50:19 GMT, Prasanta Sadhukhan <[email protected]>
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
