On Mon, 15 Sep 2025 06:24:15 GMT, Tejesh R <[email protected]> wrote:

> Test is failing frequently on mach5 machines. I have fixed it with 
> stabilizations and moved the frame to center of the screen. After the fix 
> several runs were made on mach5 and no failures were seen.

Please verify whether adding only one set of `waitForIdle` and `delay` after 
dragging is fully finished is enough.

Revert all the formatting changes to lines that you didn't change otherwise.

test/jdk/java/awt/dnd/DragSourceMotionListenerTest.java line 187:

> 185:             }
> 186:             robot.waitForIdle();
> 187:             robot.delay(100);

Will the test be as stable without these two? Dragging isn't stopped here.

In other words, isn't adding `waitForIdle` after dragging is fully finished at 
lines 198–199 but before releasing the mouse button and the <kbd>Ctrl</kbd> key.

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

Changes requested by aivanov (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27283#pullrequestreview-3229258592
PR Review Comment: https://git.openjdk.org/jdk/pull/27283#discussion_r2352000209

Reply via email to