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