8167160: [TEST_BUG][PIT] failure of javax/swing/JRadioButton/8033699/bug8033699.java

2016-11-09 Thread Avik Niyogi
Hi All, Kindly review the proposed webrev for JDK9. Bug: https://bugs.openjdk.java.net/browse/JDK-8167160 Webrev: http://cr.openjdk.java.net/~aniyogi/8167160/webrev.00/ Issue: The test

Re: 8167160: [TEST_BUG][PIT] failure of javax/swing/JRadioButton/8033699/bug8033699.java

2016-11-09 Thread Prasanta Sadhukhan
looks good to me. Regards Prasanta On 11/9/2016 1:36 PM, Avik Niyogi wrote: Hi All, Kindly review the proposed webrev for JDK9. *Bug: https://bugs.openjdk.java.net/browse/JDK-8167160* *Webrev: http://cr.openjdk.java.net/~aniyogi/8167160/webrev.00/

Re: 8167160: [TEST_BUG][PIT] failure of javax/swing/JRadioButton/8033699/bug8033699.java

2016-11-09 Thread Sergey Bylokhov
On 09.11.16 11:06, Avik Niyogi wrote: *Bug: https://bugs.openjdk.java.net/browse/JDK-8167160* *Webrev: http://cr.openjdk.java.net/~aniyogi/8167160/webrev.00/* *Issue: *The test case : javax/swing/JRadioButton/8033699/bug8033699.java fails on OS X *Cause: * The test case does not apply for OS X

Re: [9] Review request for 8075084: https://bugs.openjdk.java.net/browse/JDK-8075084

2016-11-09 Thread Alexandr Scherbatiy
On 10/25/2016 4:24 PM, Semyon Sadetsky wrote: Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8075084 webrev: http://cr.openjdk.java.net/~ssadetsky/8075084/webrev.00/ When a modal dialog is shown in scroll bar button click listener it blocks all events target

Re: 8167160: [TEST_BUG][PIT] failure of javax/swing/JRadioButton/8033699/bug8033699.java

2016-11-09 Thread Avik Niyogi
Arrow buttons for traversing a radio button group is not the expected OS X behaviour. In OS X in default OS X apps, for radio button focus traversing to work, custom actions must be set using System Preferences. > On 09-Nov-2016, at 6:51 pm, Sergey Bylokhov > wrote: > > On 09.11.16 11:06, Avi

Re: [9] Review request for 8160087: Change IOOBE to warning in the scenarios when it had not being thrown before the JDK-8078514

2016-11-09 Thread Semyon Sadetsky
please review the updated webrev: http://cr.openjdk.java.net/~ssadetsky/8160087/webrev.02/ system property was removed. --Semyon On 7/21/2016 4:40 PM, Semyon Sadetsky wrote: On 7/21/2016 2:51 PM, Alexandr Scherbatiy wrote: On 7/20/2016 4:46 PM, Semyon Sadetsky wrote: On 7/20/2016 2:29