On Tue, 2 Jul 2024 22:12:45 GMT, Alisen Chung <ach...@openjdk.org> wrote:
>> Ravi Gupta has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8333403: Review comments fixed
>
> test/jdk/java/awt/event/ComponentEvent/ComponentEventTest.java line 171:
>
>> 169: EventQueue.invokeAndWait(() -> {
>> 170: frame.dispose();
>> 171: frame.setVisible(true);
>
> I'm not really understanding this part. Why do you dispose the frame then
> setVisible here?
Thanks to providing the review comments here I want to revalidate the screen
resources by dispose and setVisible the frame before the frame ICONIFIED test.
These lines are not required , After removing these lines test also works fine.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19521#discussion_r1665496123
