On Tue, 22 Apr 2025 05:19:42 GMT, Tejesh R <t...@openjdk.org> wrote:
>> Open source these AWT Menu tests:
>>
>> java/awt/Menu/MenuVisibilityTest/MenuVisibilityTest.java
>> java/awt/Menu/MenuActionEventTest/MenuActionEventTest.java
>> java/awt/Menu/RmInHideTest/RmInHideTest.java
>> java/awt/Menu/SetShortCutTest/SetShortCutTest.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/SetShortCutTest.java line 106:
> 104: }
> 105:
> 106: public void actionPerformed(ActionEvent event) {
Add `@Override` here as well
test/jdk/java/awt/Menu/SetShortCutTest.java line 113:
> 111: }
> 112:
> 113: //Click on the "Stuff" menu to remove the "first" menu item
Suggestion:
// Click on the "Stuff" menu to remove the "first" menu item
test/jdk/java/awt/Menu/SetShortCutTest.java line 116:
> 114: else if (event.getSource() == item2) {
> 115: //If the item has not been removed from the menu,
> 116: //then remove "First" from the "File" menu
Suggestion:
// If the item has not been removed from the menu,
// then remove "First" from the "File" menu
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24649#discussion_r2053345722
PR Review Comment: https://git.openjdk.org/jdk/pull/24649#discussion_r2053346577
PR Review Comment: https://git.openjdk.org/jdk/pull/24649#discussion_r2053346858
