On Mon, 15 Sep 2025 19:15:46 GMT, Alexey Ivanov <[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.
>
> test/jdk/java/awt/dnd/DragSourceMotionListenerTest.java line 178:
> 
>> 176:             robot.keyPress(KeyEvent.VK_CONTROL);
>> 177:             robot.waitForIdle();
>> 178:             robot.delay(100);
> 
> Won't `robot.setAutoWaitForIdle(true)` resolve the problem then? Or 
> increasing the value passed to `robot.setAutoDelay`?

Either of them didn't solve the problem. Hence I have add delay.

> test/jdk/java/awt/dnd/DragSourceMotionListenerTest.java line 191:
> 
>> 189:             for (int i = 0; i < 10; i++) {
>> 190:                 robot.mouseMove(srcPoint.x, srcPoint.y++);
>> 191:             }
> 
> Doesn't this block of `mouseMove` require `waitForIdle` too?

Without this the test is passing for multiple test runs, so I don't think its 
required.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27283#discussion_r2351141300
PR Review Comment: https://git.openjdk.org/jdk/pull/27283#discussion_r2351144621

Reply via email to