RFR: 8353446: Open source several AWT Menu tests - Batch 2 [v5]

Abhishek Kumar abhiscxk at openjdk.org
Tue Apr 22 06:22:59 UTC 2025


On Tue, 22 Apr 2025 06:14:01 GMT, Tejesh R <tr at 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


More information about the client-libs-dev mailing list