On Fri, 26 Sep 2025 07:04:57 GMT, Prasanta Sadhukhan <[email protected]> 
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
>
> Prasanta Sadhukhan has updated the pull request with a new target base due to 
> a merge or a rebase. The pull request now contains eight commits:
> 
>  - PL updation
>  - Merge master
>  - Added mac ARM JBS
>  - PL updation
>  - Merge branch 'master' of https://git.openjdk.java.net/jdk into JDK-8273617
>  - Remove displayMode reset
>  - EDT fix, timeout reduced
>  - 8273617: UninitializedDisplayModeChangeTest.java times out on macOS 12

I tested the fix on both a retina display and on an external display, but could 
not recreate the hang. AFAIK, this seems to work and LGTM. This is on MacOS 
15.7.1 on M4.

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

Marked as reviewed by dnguyen (Committer).

PR Review: https://git.openjdk.org/jdk/pull/27365#pullrequestreview-3394604960

Reply via email to