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