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

[Alexey Ivanov]

On Fri, 3 Oct 2025 11:05:06 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Test frame is changed to Red for Mouse PRESSED and MOUSE_OVER state but it seem time is too less before it is moved to Green when mouse press is released so it retrieves Green instead of Red.
>> Also, the pixel color retrieval point is conflicting with mouse cursor position, which gets picked up during `robot.createSceenCapture` execution so retrieval point is moved slightly up instead of down.
>> 
>> 100 iterations of the test passed in all platforms.
>
> 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.

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

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