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

Tejesh R tr at openjdk.org
Tue Apr 22 06:42:37 UTC 2025


On Tue, 22 Apr 2025 06:20:42 GMT, Abhishek Kumar <abhiscxk at openjdk.org> wrote:

>> 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.

Updated.

> 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.

Updated.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/24681#discussion_r2053426663
PR Review Comment: https://git.openjdk.org/jdk/pull/24681#discussion_r2053426538


More information about the client-libs-dev mailing list