RFR: 8328110: Allow simultaneous use of PassFailJFrame with split UI and additional windows

Harshitha Onkar honkar at openjdk.org
Thu Mar 14 00:44:44 UTC 2024


On Wed, 13 Mar 2024 16:55:08 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:

> It is currently blocked with an artificial check:
> 
> 
> if (panelCreator != null) {
>     if (splitUI && (testWindows != null || windowListCreator != null)) {
>         // TODO Is it required? We can support both
>         throw new IllegalStateException("Split UI is not allowed "
>                                         + "with additional windows");
>     }
> }
> 
> 
> We have a number of manual tests, which has some text area in the instruction window to print a feedback from the test to be evaluated by a tester.
> 
> Removing this check allows us to implement this.
> We can consider making a special methods for this later.
> 
> This is required for the #18250

Couple of similar tests can benefit by integrating this change of using testUI & splitUI in combination.

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

Marked as reviewed by honkar (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18281#pullrequestreview-1935452653


More information about the client-libs-dev mailing list