Looks good to me.
Regards, Rajeev Chamyal From: Avik Niyogi Sent: 08 September 2016 11:00 To: Rajeev Chamyal Cc: Alexandr Scherbatiy; swing-dev@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.scherba...@oracle.com"alexandr.scherba...@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.cham...@oracle.com"rajeev.cham...@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@openjdk.java.net"swing-dev@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