RFR: 8273617: UninitializedDisplayModeChangeTest.java times out on macOS 12
Phil Race
prr at openjdk.org
Fri Sep 19 05:31:14 UTC 2025
On Fri, 19 Sep 2025 02:55:34 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> 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)
I think you may have missed the point as to why this test is unstable.
Look at all the code running on the EDT which constructs the UI that should be either not run there or should be in separate calls.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27365#discussion_r2361816592
More information about the client-libs-dev
mailing list