On Fri, 3 Oct 2025 11:05:06 GMT, Prasanta Sadhukhan <[email protected]> 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
