Re: 8163169: [PIT][TEST_BUG] fix to JDK-8161470 doesn't work

2016-08-17 Thread Rajeev Chamyal
Looks ok to me.

 

Regards,

Rajeev Chamyal

 

From: Alexandr Scherbatiy 
Sent: 17 August 2016 12:26
To: Avik Niyogi; Rajeev Chamyal; swing-dev@openjdk.java.net
Subject: Re:  8163169: [PIT][TEST_BUG] fix to JDK-8161470 doesn't 
work

 

The fix looks good to me.

Thanks,
Alexandr.

On 8/17/2016 9:48 AM, Avik Niyogi wrote:

Hi All,

 

Kindly review the fix for JDK9. 

 

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

 

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

 

Issue: The test javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java 
fails on OS X

 for PIT machines even though it works fine for local machine.

 

Cause:  As pointed out, this is an issue with the robot autoDelay.

 

Fix: Appropriate Robot methods were used to fix this with enough delays added 
to work on PIT machines.

 

With Regards,

Avik Niyogi

 


Re: 8163169: [PIT][TEST_BUG] fix to JDK-8161470 doesn't work

2016-08-16 Thread Alexandr Scherbatiy

The fix looks good to me.

Thanks,
Alexandr.

On 8/17/2016 9:48 AM, Avik Niyogi wrote:

Hi All,

Kindly review the fix for JDK9.

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

*Webrev: **http://cr.openjdk.java.net/~aniyogi/8163169/webrev.00/* 



*Issue: *The test 
javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java fails on OS X

 for PIT machines even though it works fine for local machine.

*Cause: * As pointed out, this is an issue with the robot autoDelay.

*Fix:* Appropriate Robot methods were used to fix this with enough 
delays added to work on PIT machines.


With Regards,
Avik Niyogi




8163169: [PIT][TEST_BUG] fix to JDK-8161470 doesn't work

2016-08-16 Thread Avik Niyogi
Hi All,

Kindly review the fix for JDK9.

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


Webrev: http://cr.openjdk.java.net/~aniyogi/8163169/webrev.00/ 


Issue: The test javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java 
fails on OS X
 for PIT machines even though it works fine for local machine.

Cause:  As pointed out, this is an issue with the robot autoDelay.

Fix: Appropriate Robot methods were used to fix this with enough delays added 
to work on PIT machines.

With Regards,
Avik Niyogi