On Wed, 29 Nov 2023 18:32:46 GMT, Harshitha Onkar <[email protected]> wrote:

>> Added delays to stabilize test, test passes 50 times consecutively
>
> test/jdk/java/awt/Window/Grab/GrabTest.java line 221:
> 
>> 219:         robot.delay(100);
>> 220:         robot.mouseRelease(InputEvent.BUTTON1_MASK);
>> 221:         Util.waitForIdle(robot);
> 
> Can be replaced with BUTTON1_DOWN_MASK.
> 
> Has the fix been tested on macOS 14 ?

That (about macOS 14) is the important question. Why is it so unstable on macOS 
14 but fine everywhere else ?

Also can't you use "setAutoDelay(100)" instead of adding all these one-off 
delays ?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16867#discussion_r1409742126

Reply via email to