RFR: 8316149: Open source several Swing JTree JViewport KeyboardManager tests

Damon Nguyen dnguyen at openjdk.org
Tue Sep 19 16:41:42 UTC 2023


On Tue, 19 Sep 2023 07:46:20 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> These are the tests being converted:
>> 
>> javax/swing/JTree/4696499/bug4696499.java
>> javax/swing/JTree/5039542/bug5039542.java
>> javax/swing/JViewport/4546474/bug4546474.java
>> javax/swing/JViewport/4677611/bug4677611.java
>> javax/swing/KeyboardManager/4345798/bug4345798.java
>
> test/jdk/javax/swing/KeyboardManager/bug4345798.java line 50:
> 
>> 48:     private static JFrame f;
>> 49:     private static JButton b;
>> 50:     private static JMenu menu;
> 
> guess it can be local var

JFrame needs to be disposed in the finally block. JButton is needed in the TestActionListener. And the JMenu is needed for the getLocationOnScreen call on line 84. I believe these need to be static instead of local vars.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15802#discussion_r1330419106


More information about the client-libs-dev mailing list