On Tue, 16 Sep 2025 11:01:15 GMT, Alexey Ivanov <[email protected]> wrote:

>> Tejesh R has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Review fix
>
> 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.

Adding waitForIdle and delay at line 176 makes the test stable. I have removed 
all the changes and test passes multiple runs at CI without failure.

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

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

Reply via email to