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

[Alisen Chung]

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 Updated changes look good. Missing comma needs to be added to copyright as suggested by @lawrence-andrew earlier.
> 
> Does CI Testing look good?

CI testing is green

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

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