On Tue, 22 Apr 2025 06:14:01 GMT, Tejesh R <t...@openjdk.org> 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

Reply via email to