RFR: 8316389: Open source few AWT applet tests [v6]

Phil Race prr at openjdk.org
Mon Sep 25 18:03:54 UTC 2023


On Mon, 25 Sep 2023 17:33:35 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Alexander Zvegintsev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review comments
>
> test/jdk/java/awt/Frame/WindowMoveTest.java line 57:
> 
>> 55:                 frame.dispatchEvent(new WindowEvent(frame, WindowEvent.WINDOW_CLOSING)));
>> 56: 
>> 57:         if (!WindowMove.latch.await(10, TimeUnit.SECONDS)) {
> 
> I think 10 seconds is too long… We usually use 2 seconds for various delays.

I agree, If it doesn't happen in much less than 10 seconds probably it'll never happen.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15787#discussion_r1336224236


More information about the client-libs-dev mailing list