RFR: 8288444: Remove the workaround for frame.pack() in ModalDialogTest [v2]

Harshitha Onkar honkar at openjdk.org
Thu Jun 30 00:05:26 UTC 2022


> Previously frame.pack() was called twice as a workaround  AWT frame sizing issue in this particular test case on Windows. This issue was fixed **[JDK-8265586](https://bugs.openjdk.org/browse/JDK-8265586)** and hence the workaround is being removed from this test.
> 
> PassFailJFrame is being used as the test framework. 
> 
> **Proposed change related to PassFailJFrame Test framework**
> 
> Instead of creating separate methods to handle AWT Windows as well as Frames, method signatures - [**PassFailJFrame.positionTestFrame()**](https://github.com/openjdk/jdk/blob/53b37fe1535388eb14e04c620a6b0118ed8884a0/test/jdk/java/awt/regtesthelpers/PassFailJFrame.java#L271) and [**PassFailJFrame.addTestFrame()**](https://github.com/openjdk/jdk/blob/53b37fe1535388eb14e04c620a6b0118ed8884a0/test/jdk/java/awt/regtesthelpers/PassFailJFrame.java#L294) can be changed to take in Window as input parameter instead of Frame, since Window is a superclass, this would allow the test frame to be a Window/Frame/Dialog and not be restricted to a Frame, additionally would allow proper disposal of windows added to the PassFailJFrame.

Harshitha Onkar has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:

 - Merge branch 'master' into ModalTestGBr_8288444
 - ModalDialogTest changes

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/9286/files
  - new: https://git.openjdk.org/jdk/pull/9286/files/e0f3fddd..6f9e78a1

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=9286&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9286&range=00-01

  Stats: 55872 lines in 909 files changed: 33563 ins; 10740 del; 11569 mod
  Patch: https://git.openjdk.org/jdk/pull/9286.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9286/head:pull/9286

PR: https://git.openjdk.org/jdk/pull/9286



More information about the client-libs-dev mailing list