On Mon, 14 Aug 2023 21:00:56 GMT, Alisen Chung <[email protected]> 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
