Looks good to me.

Best regards,
Alexander Kouznetsov

On 2/13/2017 10:09 PM, Vikrant Agarwal wrote:

Hi Sergey,

Thanks, I have removed JemmyExt from the new webrev:

http://cr.openjdk.java.net/~vagarwal/8174161/webrev_2/ <http://cr.openjdk.java.net/%7Evagarwal/8174161/webrev_2/>

Best Regards,

Vikrant

*From:*Vikrant Agarwal [mailto:[email protected]]
*Sent:* Tuesday, February 14, 2017 11:38 AM
*To:* Sergey Bylokhov
*Cc:* '[email protected]'; Aleksandre Iline; Alexander Kouznetsov
*Subject:* RE: <Swing Dev> Review Request for JDK-8174161

Hi Sergey,

Thanks, I have removed JemmyExt from the new webrev.

Best Regards,

Vikrant

*From:*Sergey Bylokhov
*Sent:* Monday, February 13, 2017 11:55 PM
*To:* Vikrant Agarwal
*Cc:* [email protected] <mailto:[email protected]>; Aleksandre Iline; Alexander Kouznetsov
*Subject:* Re: <Swing Dev> Review Request for JDK-8174161

Looks fine, but I guess the change in JemmyExt is not needed.

    Hi All,

    Please review the following:

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

    Webrev Link:http://cr.openjdk.java.net/~vagarwal/8174161/webrev_1/
    <http://cr.openjdk.java.net/%7Evagarwal/8174161/webrev_1/>

    Summary: This is a new test for automated testing of SwingSet
    DialogDemo using Jemmy.

    Please see the bug for description of the scenario’s automated.

    Best Regards,

    Vikrant


Reply via email to