[jdk8u-dev] RFR: 8160767: [TEST_BUG] java/awt/Frame/MaximizedToIconified/MaximizedToIconified.java

[Taizo Kurashige]

On Thu, 13 Mar 2025 01:56:30 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> Hi all,
>> 
>> This is a backport of JDK-8160767: [TEST_BUG] java/awt/Frame/MaximizedToIconified/MaximizedToIconified.java
>> 
>> Original patch applies cleanly to 8u (only path changes).
>> 
>> Testing: java/awt/Frame/MaximizedToIconified/MaximizedToIconified.java on RHEL9.2
>> 
>> Thanks.
>
> jdk/test/java/awt/Frame/MaximizedToIconified/MaximizedToIconified.java line 55:
> 
>> 53:     private static void checkState(Frame frame, int state) {
>> 54:         frame.setExtendedState(state);
>> 55:         robot.waitForIdle();
> 
> I'm not sure this will be enough for jdk8. https://bugs.openjdk.org/browse/JDK-8056911 was not ported to jdk8, so Robot.waitForIdle() uses a "dummy" implementation.

Sorry, I wasn't aware that Robot.waitForIdle() uses a dummy implementation.
I thought it was necessary to apply [JDK-8056911](https://bugs.openjdk.org/browse/JDK-8056911) first, but backporting [JDK-8056911](https://bugs.openjdk.org/browse/JDK-8056911) has been rejected in the past because changing the behavior of public methods is undesirable.
I will consider whether I should backport [JDK-8056911](https://bugs.openjdk.org/browse/JDK-8056911). If the [JDK-8056911](https://bugs.openjdk.org/browse/JDK-8056911) backport is not appropriate, I will implement another fix in this PR.

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

PR Review Comment: https://git.openjdk.org/jdk8u-dev/pull/637#discussion_r2009375866