On Tue, 22 Apr 2025 06:14:01 GMT, Tejesh R <[email protected]> wrote:
>> Open source these AWT Menu tests:
>>
>> java/awt/Menu/OnFlyRepaintMenuTest/OnFlyRepaintMenuTest.java
>> java/awt/Menu/DestroyMenuTest/DestroyMenuTest.java
>> java/awt/Menu/MenuAddRemoveCrash/MenuAddRemoveCrash.java
>> java/awt/Menu/MenuZOrderTest/MenuZOrderTest.java
>
> Tejesh R has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Updated review comments
test/jdk/java/awt/Menu/MenuZOrderTest.java line 33:
> 31: */
> 32:
> 33: import java.awt.BorderLayout;
Can be removed.
test/jdk/java/awt/Menu/MenuZOrderTest.java line 73:
> 71: MenuItem mi1 = new MenuItem("Menu Item 1");
> 72:
> 73: mf.setLayout(new BorderLayout());
Don't think it is required.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24681#discussion_r2053404743
PR Review Comment: https://git.openjdk.org/jdk/pull/24681#discussion_r2053404693
