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 > <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: http://cr.openjdk.java.net/~aniyogi/8163274/webrev.01/ >> <http://cr.openjdk.java.net/%7Eaniyogi/8163274/webrev.01/> >> >> With Regards, >> Avik Niyogi >> >>> On 07-Sep-2016, at 11:03 am, Rajeev Chamyal <rajeev.cham...@oracle.com >>> <mailto: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; swing-dev@openjdk.java.net >>> <mailto: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 >>> <https://bugs.openjdk.java.net/browse/JDK-8163274> >>> >>> Webrev: http://cr.openjdk.java.net/~aniyogi/8163274/webrev.00/ >>> <http://cr.openjdk.java.net/%7Eaniyogi/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 >>> >> >