<AWT Dev> [9] Review Request: 8004693 TEST_BUG: java/awt/KeyboardFocusmanager/DefaultPolicyChange/DefaultPolicyChange_Swing.java fails

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Mon May 30 16:39:44 UTC 2016


Hello.
Please review the fix for jdk9.

The test DefaultPolicyChange_Swing.java has two issues:
  - It uses invokeLater(), so the test usually pass before the code is 
executed on the EDT, because the main thread completes before.
  - The test fetches the FocusTraversalPolicy from the current 
KeyboardFocusManager. But default FocusTraversalPolicy can be changed 
during the Swing initialization(JDK-7125044). The test should save the 
state before setDefaultFocusTraversalPolicy() but after the Swing 
initialization, and validate that the FocusTraversalPolicy was not 
changed for windows which were already shown.

The fix proposed in the CR is applied + small cleanup(regtesthelpers 
removed and InvokeAndWait is used instead of InvokeLater+realSync)

Bug: https://bugs.openjdk.java.net/browse/JDK-8004693
Webrev can be found at: http://cr.openjdk.java.net/~serb/8004693/webrev.01

-- 
Best regards, Sergey.


More information about the awt-dev mailing list