RFR: 8273617: UninitializedDisplayModeChangeTest.java times out on macOS 12
Prasanta Sadhukhan
psadhukhan at openjdk.org
Fri Sep 19 02:58:30 UTC 2025
On Fri, 19 Sep 2025 01:14:33 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
>> Test used to timeout even though it seems the test passed..Increased the timeout to a safe value as sometimes it shows elapsed time to timeout > 300secs in macOS in CI and also ensured the wait-time for child process to execute the test is not been waiting endlessly.
>> Also ensured the original display mode is resetted after the test to prevent affecting following tests.
>>
>> Tried 100 iterations of the fix on all platforms which is ok
>
> test/jdk/java/awt/FullScreen/UninitializedDisplayModeChangeTest/DisplayModeChanger.java line 69:
>
>> 67: f.dispose();
>> 68: }
>> 69: gd.setDisplayMode(currentDM);
>
> Should not this be already reset by the gd.setFullScreenWindow(null)?
I dont see it does..It sets the bounds as I see, it does not reset the display mode..
but say, even though it does, will it be a problem as this will be a no-op (its like double free)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27365#discussion_r2361614626
More information about the client-libs-dev
mailing list