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