The fix looks good to me.
Thanks,
Alexandr.
On 7/20/2016 9:34 AM, Rajeev Chamyal wrote:
Looks fine to me.
Regards,
Rajeev Chamyal
*From:*Avik Niyogi
*Sent:* 20 July 2016 11:52
*To:* Rajeev Chamyal
*Cc:* Praveen Srivastava; Alexander Scherbatiy; [email protected]
*Subject:* Re: <Swing Dev> 8161470: [TEST_BUG] Failure
javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java
Hi Rajeev,
Updated on same webrev due to single line change.
On 20-Jul-2016, at 11:41 am, Rajeev Chamyal
<[email protected] <mailto:[email protected]>> wrote:
Hello Avik,
Line 67 can be removed from test.
Regards,
Rajeev Chamyal
*From:*Avik Niyogi
*Sent:*20 July 2016 11:30
*To:*Rajeev Chamyal
*Cc:*Praveen Srivastava; Alexandr Scherbatiy;
[email protected] <mailto:[email protected]>
*Subject:*Re: <Swing Dev> 8161470: [TEST_BUG] Failure
javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java
A gentle reminder, please review my code changes.
With Regards,
Avik Niyogi
On 19-Jul-2016, at 1:31 pm, Alexandr Scherbatiy
<[email protected]
<mailto:[email protected]>> wrote:
The fix looks good to me.
Thanks,
Alexandr.
On 7/19/2016 9:45 AM, Avik Niyogi wrote:
Hi All,
Kindly review the fix for JDK9.
*Bug: https://bugs.openjdk.java.net/browse/JDK-8161470*
*Webrev:
http://cr.openjdk.java.net/~aniyogi/8161470/webrev.00/
<http://cr.openjdk.java.net/%7Eaniyogi/8161470/webrev.00/>*
*Issue: *This test consistently (12/12) fails on OS X
10.10 and also fails on Ubuntu 14.04.
*Cause: * Due to quirk in robot.delay the UI was not
getting appropriate behaviour response.
*Fix:* Appropriate Robot methods were used to fix this.
With Regards,
Avik Niyogi