<Swing Dev> Review for 8132770: Test javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java fails in MacOSX

[Avik Niyogi]

Hi All,

The bug fix has been updated as per the comments.
http://cr.openjdk.java.net/~rchamyal/avik/8132770/webrev.01/ <http://cr.openjdk.java.net/~rchamyal/avik/8132770/webrev.01/>


With Regards,
Avik Niyogi

> On 13-Nov-2015, at 9:46 am, Avik Niyogi <avik.niyogi at oracle.com> wrote:
> 
> Hi Rajeev,
> 
> Will fix and create webrev.01 with inputs provided. Also, we have verified on Mac OS X, Windows and Ubuntu.
> 
> With Regards,
> Avik Niyogi
> 
>> On 12-Nov-2015, at 4:52 pm, Rajeev Chamyal <rajeev.chamyal at oracle.com <mailto:rajeev.chamyal at oracle.com>> wrote:
>> 
>> Hello Avik,
>>  
>> 1)      In the tryLookAndFeel exception handling is skipped. createUI and other calls should be skipped in case of exception..
>> 2)      What all platforms you have tested the bug?
>>  
>> Regards,
>> Rajeev Chamyal
>>  
>> From: Avik Niyogi 
>> Sent: 04 November 2015 14:51
>> To: Rajeev Chamyal; Alexander Scherbatiy; swing-dev at openjdk.java.net <mailto:swing-dev at openjdk.java.net>
>> Subject: Review for 8132770: Test javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java fails in MacOSX
>>  
>> Hi All,
>>  
>> Kindly review the fix for JDK9.
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8132770 <https://bugs.openjdk.java.net/browse/JDK-8132770>
>>  
>> Webrev: http://cr.openjdk.java.net/~rchamyal/avik/8132770/webrev.00/ <http://cr.openjdk.java.net/~rchamyal/avik/8132770/webrev.00/>
>>  
>> Issue: Test javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java fails in MacOSX where non-eligible Look and Feels are not taken into account.
>>  
>> Cause: TestCase flow for non-Basic JRadioButton UI such as Aqua, Nimbus and GTK have different focus traversal track as compared to the rest and require different test case flow with regards to focus owner.
>>  
>> Fix: Modified test case with appropriate deviant test case for Look and Feels which are not eligible for focus traversal as expected previously in test file.
>>  
>> With Regards,
>> Avik Niyogi
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20151116/7b586b52/attachment.html>