Looks good to me.
Regards, Rajeev Chamyal From: Avik Niyogi Sent: 24 August 2016 14:14 To: Rajeev Chamyal Cc: Alexander Scherbatiy; swing-dev@openjdk.java.net Subject: Re: <Swing Dev> 8163161: [PIT][TEST_BUG] increase timeout in javax/swing/plaf/nimbus/8057791/bug8057791.java Hi All, A gentle reminder, please review my code change. With Regards, Avik Niyogi On 23-Aug-2016, at 2:28 pm, Alexander Scherbatiy <HYPERLINK "mailto:alexandr.scherba...@oracle.com"alexandr.scherba...@oracle.com> wrote: The fix looks good to me. Thanks, Alexandr. On 23/08/16 10:38, Avik Niyogi wrote: Hi All, Kindly review the fix for JDK9. Bug: https://bugs.openjdk.java.net/browse/JDK-8163161 Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Eaniyogi/8163161/webrev.00/"http://cr.openjdk.java.net/~aniyogi/8163161/webrev.00/ Issue: This test javax/swing/plaf/nimbus/8057791/bug8057791.java times out Cause: No timeout delay was provided so it would fail on other systems. Fix: Appropriate changes for timeout were added. With Regards, Avik Niyogi