RFR: 8315484: java/awt/dnd/RejectDragDropActionTest.java timed out [v3]

[Alexey Ivanov]

On Tue, 3 Oct 2023 22:36:15 GMT, Damon Nguyen <dnguyen at openjdk.org> wrote:

>> test/jdk/java/awt/dnd/RejectDragDropActionTest.java line 91:
>> 
>>> 89:             robot.setAutoWaitForIdle(true);
>>> 90:             robot.waitForIdle();
>>> 91:             robot.delay(DELAY_TIME);
>> 
>> To reduce the initial delay, you may override `Frame.paint` and `countDown` a latch after calling `super.paint`. Once the latch is released, you create robot and start testing.
>> 
>> `DELAY_TIME` could be set to `500` or even less.
>
> I reduced the `DELAY_TIME` to 500. Thanks for the suggestion. I got used to setting the default delay times to 1000 for these types of tests, and am trying to break the habit and reduce delays further to 500 where possible.

Well, I meant that you could reduce the delay if you add the latch.

However, 500ms should be enough now. Windows Vista and Windows 7 had long window animations when a window was shown on the screen, I believe this is where the delay of 1 second comes from. Windows 10 and 11 still have this animation but it's quicker.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16018#discussion_r1345658202