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");
>>         }
>>     }
>> });
>
>> @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?

@alisenchung  I think another option would be to override `componentShown()` of  ComponentListener and check the size of the frame returned when it is made visible. With original code, I think it would show 200x200 and with the fix , the maximized frame size. I haven't tested this option though.


@Override
 public void componentShown(ComponentEvent e) {
        System.out.println(e.getComponent().getSize());
    }

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

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