On Fri, 3 Oct 2025 14:30:29 GMT, Alexey Ivanov <[email protected]> wrote:

>> 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.

Reducing delay further causes failure in CI so I will keep it

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

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

Reply via email to