On Wed, 18 Sep 2024 15:39:04 GMT, Abhishek Kumar <abhis...@openjdk.org> wrote:

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

Updated the test to set the test frames visible, `setVisible(true)`;

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

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

Reply via email to