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; swing-dev@openjdk.java.net
*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
    <rajeev.cham...@oracle.com <mailto:rajeev.cham...@oracle.com>> 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;
    swing-dev@openjdk.java.net <mailto:swing-dev@openjdk.java.net>
    *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
        <alexandr.scherba...@oracle.com
        <mailto:alexandr.scherba...@oracle.com>> 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


Reply via email to