<Swing Dev> [12] Review Request: 8198398 Test javax/swing/JColorChooser/Test6199676.java fails in mach5

[Krishna Addepalli]

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: swing-dev at openjdk.java.net
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.