On Sun, 29 Sep 2024 23:54:13 GMT, Damon Nguyen <[email protected]> wrote:
> Following tests are added as part of this PR:
>
> /java/awt/MenuBar/MenuBarOnDisabledFrame/MenuBarOnDisabledFrame.java (Manual
> with PassFailJFrame)
> /java/awt/MenuBar/MenuBarAddRemoveTest/MenuBarAddRemoveTest.java (Manual with
> PassFailJFrame)
> /java/awt/MenuBar/SetHelpMenuTest/SetHelpMenuTest.java (Manual with
> PassFailJFrame)
> /java/awt/MenuBar/MenuBarVisuals/MenuBarVisuals.java (Manual with
> PassFailJFrame)
test/jdk/java/awt/MenuBar/MenuBarAddRemoveTest/MenuBarAddRemoveTest.java line
28:
> 26: * @bug 4028130 4112308
> 27: * @summary Disabling a frame does not disable the menus on the frame, on
> 28: * solaris/linux
seems summary is wrong for this test...
test/jdk/java/awt/MenuBar/MenuBarVisuals/MenuBarVisuals.java line 63:
> 61:
> 62: private static Frame createUI() {
> 63: Frame f = new Frame("Test frame");
meaningful frame name can be given...
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21251#discussion_r1780359098
PR Review Comment: https://git.openjdk.org/jdk/pull/21251#discussion_r1780359591
