<Swing Dev> 8163274: [TEST_BUG][macosx] apparent regression: javax/swing/JColorChooser/Test7194184.java

[Rajeev Chamyal]

Looks good to me.

 

Regards,

Rajeev Chamyal

 

From: Avik Niyogi 
Sent: 08 September 2016 11:00
To: Rajeev Chamyal
Cc: Alexandr Scherbatiy; swing-dev at openjdk.java.net
Subject: Re: <Swing Dev> 8163274: [TEST_BUG][macosx] apparent regression: javax/swing/JColorChooser/Test7194184.java

 

A gentle reminder, please review code changes as indicated in new webrev.

 

With Regards,

Avik Niyogi

 

On 07-Sep-2016, at 9:50 pm, Alexandr Scherbatiy <HYPERLINK "mailto:alexandr.scherbatiy at oracle.com"alexandr.scherbatiy at oracle.com> wrote:

 

The fix looks good to me.

Thanks,
Alexandr.

On 9/7/2016 9:02 AM, Avik Niyogi wrote:

Hi All, 

Kindly review the updated fix for JDK9 with new inputs incorporated.

Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Eaniyogi/8163274/webrev.01/"http://cr.openjdk.java.net/~aniyogi/8163274/webrev.01/

 

With Regards,

Avik Niyogi

 

On 07-Sep-2016, at 11:03 am, Rajeev Chamyal <HYPERLINK "mailto:rajeev.chamyal at oracle.com"rajeev.chamyal at oracle.com> wrote:

 

Looks good to me.

 

Regards,

Rajeev Chamyal

 

From: Alexandr Scherbatiy 
Sent: 06 September 2016 16:16
To: Avik Niyogi; Rajeev Chamyal; HYPERLINK "mailto:swing-dev at openjdk.java.net"swing-dev at openjdk.java.net
Subject: Re: <Swing Dev> 8163274: [TEST_BUG][macosx] apparent regression: javax/swing/JColorChooser/Test7194184.java

 

The fix looks good to me.

Thanks,
Alexandr.

On 9/6/2016 9:12 AM, Avik Niyogi wrote:

Hi All,

 

Kindly review the fix for JDK9.  

 

Bug: https://bugs.openjdk.java.net/browse/JDK-8163274

 

Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Eaniyogi/8163274/webrev.00/"http://cr.openjdk.java.net/~aniyogi/8163274/webrev.00/

 

Issue: This test javax/swing/JColorChooser/Test7194184.java throws exception due to event delay.

 

Cause:  Delay for event idling was not added.

 

Fix: Appropriate changes for autoWaitForIdle() were added.

 

With Regards,

Avik Niyogi

 

 

 

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20160907/b8cc7f5c/attachment.html>