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