On Mon, 14 Aug 2023 21:00:56 GMT, Alisen Chung <ach...@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
