RFR: 8317288: [macos] java/awt/Window/Grab/GrabTest.java: Press on the outside area didn't cause ungrab [v3]

Alexander Zvegintsev azvegint at openjdk.org
Tue Dec 5 07:00:35 UTC 2023


On Tue, 5 Dec 2023 06:26:03 GMT, Alisen Chung <achung at openjdk.org> wrote:

>> Added delays to stabilize test, test passes 50 times consecutively
>
> Alisen Chung has updated the pull request incrementally with one additional commit since the last revision:
> 
>   updated test

test/jdk/java/awt/Window/Grab/GrabTest.java line 122:

> 120:             f.setVisible(true);
> 121:             w.setVisible(true);
> 122:             robot.delay(500);

This delay blocks the UI thread because it is called on EDT.
It should be called outside of the `SwingUtilities.invokeAndWait` block, e.g. before the `test()` call.

There are also other `setVisible` calls in this test that require similar delay.
Especially in cases where we try to click on a component right after showing a frame.

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

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


More information about the client-libs-dev mailing list