Hi Sergey, I think you need to wrap the code in run() method inside a try/finally block, since if setLookAndFeel function throws exception, the frame will not be disposed. Other than that, it looks fine.
Thanks, Krishna -----Original Message----- From: Sergey Bylokhov Sent: Saturday, November 10, 2018 5:35 AM To: [email protected] Subject: <Swing Dev> [12] Review Request: 8198398 Test javax/swing/JColorChooser/Test6199676.java fails in mach5 Hello. Please review the fix for jdk 12. Bug: https://bugs.openjdk.java.net/browse/JDK-8198398 Webrev: http://cr.openjdk.java.net/~serb/8198398/webrev.00 Exception reported in this bug is not reproducible since we run our tests in othervm mode, but we have an opposite bug, the jtreg may report "PASS" status for the tests even before the "Runnable" will start to execute. Change description: - bug6735286.java: invokeLater() was replaced by the invokeAndWait(), so the jtreg will wait till completion of the test. - Test6199676.java: the test used a sequence of invokeLater(), each executes part of test case. It was replaced by one step inside invokeAndWait(). I also confirm that the updated tests still can be used for verification of: "@bug 6199676" and "@bug 6735286" -- Best regards, Sergey.
