RFR: 8315871: Opensource five more Swing regression tests [v2]

Damon Nguyen dnguyen at openjdk.org
Thu Sep 14 21:48:39 UTC 2023


On Thu, 14 Sep 2023 15:27:48 GMT, Alexander Zuev <kizune at openjdk.org> wrote:

>> Clean up and move to open source five tests.
>> javax/swing/AncestorNotifier/4817630/bug4817630.java
>> javax/swing/BoxLayout/4191948/bug4191948.java
>> javax/swing/ComponentInputMap/4248723/bug4248723.java
>> javax/swing/DefaultBoundedRangeModel/4297953/bug4297953.java
>> javax/swing/DefaultButtonModel/4097723/bug4097723.java
>
> Alexander Zuev has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Update test/jdk/javax/swing/AncestorNotifier/4817630/bug4817630.java
>    
>    Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>
>  - Update test/jdk/javax/swing/DefaultBoundedRangeModel/4297953/bug4297953.java
>    
>    Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>

LGTM. Just move the frame dispose code blocks to EDT.

test/jdk/javax/swing/AncestorNotifier/4817630/bug4817630.java line 112:

> 110:             test.start();
> 111:         } finally {
> 112:             test.destroy();

Either destroy() or the if statement in destroy() for disposing the frame should be on EDT

test/jdk/javax/swing/BoxLayout/4191948/bug4191948.java line 76:

> 74:                 frame.setVisible(false);
> 75:                 frame.dispose();
> 76:             }

Should this if statement be run on EDT?

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

Marked as reviewed by dnguyen (Committer).

PR Review: https://git.openjdk.org/jdk/pull/15702#pullrequestreview-1627795667
PR Review Comment: https://git.openjdk.org/jdk/pull/15702#discussion_r1326547720
PR Review Comment: https://git.openjdk.org/jdk/pull/15702#discussion_r1326547278


More information about the client-libs-dev mailing list