On Sun, 21 Sep 2025 11:43:28 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 incrementally with one
> additional commit since the last revision:
>
> Remove displayMode reset
Marked as reviewed by prr (Reviewer).
test/jdk/java/awt/FullScreen/UninitializedDisplayModeChangeTest/DisplayModeChanger.java
line 59:
> 57: });
> 58: try {
> 59: Thread.sleep(1000);
Usually I would do Robot.waitForIdle() and then Robot.delay(1000)
But if you say it passed 50 times, then maybe this is enough.
-------------
PR Review: https://git.openjdk.org/jdk/pull/27365#pullrequestreview-3254462727
PR Review Comment: https://git.openjdk.org/jdk/pull/27365#discussion_r2369748802
