On Fri, 15 Dec 2023 15:16:32 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> songpv-imt has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Update InterJVMGetDropSuccessTest.java
>>   Instead of using SYNC_LOCK to synchronize accessing the listener's fields, 
>> for better code transparency, we have placed the code that accesses the 
>> listener's fields from the main thread into the AWT Event Queue.
>
> test/jdk/java/awt/dnd/InterJVMGetDropSuccessTest/InterJVMGetDropSuccessTest.java
>  line 273:
> 
>> 271:             robot.setAutoWaitForIdle(true);
>> 272:             robot.mouseMove(sourcePoint.x, sourcePoint.y);
>> 273:             Thread.sleep(50);
> 
> Suggestion:
> 
>             robot.delay(50);
> 
> Using `Robot.delay` is more common in tests, and it doesn't throw 
> `InterruptedException`.

Thank you for your suggestions. The changes have been incorporated.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16396#discussion_r1429444342

Reply via email to