On Wed, 18 Sep 2024 15:26:51 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Abhishek Kumar has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Test instruction update
>
> test/jdk/java/awt/MenuItem/LotsOfMenuItemsTest.java line 107:
> 
>> 105:             setMenuBar(mb);
>> 106:             setSize(450, 150);
>> 107:         }
> 
> The original test had `setVisible(true);` here. This is why the loop contains 
> `setVisible(false)`.
> 
> Without showing the frames, the test may not reproduce the original problem 
> reported in [JDK-4175790](https://bugs.openjdk.org/browse/JDK-4175790).

I think the testFrame's are made invisible and disposed just after creating. 
Only the last frame and first frame is visible.
After the test update, **testFrame and firstFrame is setVisible by 
PassFailJFrame**.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21029#discussion_r1765299255

Reply via email to