RFR: 8139208: [macosx] Issue with setExtendedState of JFrame [v2]

[Harshitha Onkar]

On Mon, 14 Aug 2023 21:00:56 GMT, Alisen Chung <achung at openjdk.org> wrote:

> > @alisenchung
> > An automated test can be created by adding Component Listener to the reproducer test in JBS. The following code throws Runtime Exception if the frame was visible at the original size of 200 x 200 before it was maximized.
> > ```
> > frame.addComponentListener(new ComponentAdapter() {
> >     @Override
> >     public void componentResized(ComponentEvent e) {
> >         if (e.getComponent().getSize().equals(new Dimension(200, 200))) {
> >             throw new RuntimeException("Test Failed! " +
> >                     "Original Frame size was visible before maximizing");
> >         }
> >     }
> > });
> > ```
> 
> When I run the test manually after the change I'm not able to see a small window anymore. Won't this code always fail since you listen for a resize event then check the size of the window right before the window is resized?

Resize event occurs in both cases, original and with the proposed fix. But we are throwing runtime exception when resize event occurs and if  _the frame **was visible at original size** before being maximized_ .  To see the difference please run the test (with Component Listener) with original and proposed fix.

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

PR Comment: https://git.openjdk.org/jdk/pull/15236#issuecomment-1678127812