On Tue, 3 Oct 2023 17:59:19 GMT, Damon Nguyen <dngu...@openjdk.org> wrote:
>> test/jdk/java/awt/dnd/RejectDragDropActionTest.java line 90: >> >>> 88: Robot robot = new Robot(); >>> 89: robot.waitForIdle(); >>> 90: robot.delay(DELAY_TIME); >> >> Does adding `robot.autoWaitForIdle(true)` additionally increase the >> stability of the test since it has subsequent DnD events performed by the >> robot? > > Sure, I just added it. Previously when I initially worked on this test, > setAutoWaitForIdle had a weird interaction with how the test was setup. Since > the test used a delay of 50ms between each of the movements when dragging the > selection, it caused long delays. Looks like this is resolved now, and the > method doesn't add any unreasonable delays anymore. Sounds good! ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16018#discussion_r1344525247
