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

[Sergey Bylokhov]

Hi, Krishna.

Thank you for review, the new version of the fix:
http://cr.openjdk.java.net/~serb/8198398/webrev.01

On 13/11/2018 22:25, Krishna Addepalli wrote:
> 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.
> 


-- 
Best regards, Sergey.