<Swing Dev> Review Request for JDK-8174161

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

<Swing Dev> Review Request for JDK-8174161

Vikrant Agarwal

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/

 

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 | Threaded
Open this post in threaded view
|

Re: <Swing Dev> Review Request for JDK-8174161

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


Hi All,
 
Please review the following:
 
 
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 | Threaded
Open this post in threaded view
|

Re: <Swing Dev> Review Request for JDK-8174161

Vikrant Agarwal

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: [hidden email]; 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:

 

 

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 | Threaded
Open this post in threaded view
|

Re: <Swing Dev> Review Request for JDK-8174161

Vikrant Agarwal
In reply to this post by Sergey Bylokhov

Hi Sergey,

 

Thanks, I have removed JemmyExt from the new webrev:

http://cr.openjdk.java.net/~vagarwal/8174161/webrev_2/

 

Best Regards,

Vikrant

 

From: Vikrant Agarwal [mailto:[hidden email]]
Sent: Tuesday, February 14, 2017 11:38 AM
To: Sergey Bylokhov
Cc: '[hidden email]'; 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: [hidden email]; 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:

 

 

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 | Threaded
Open this post in threaded view
|

Re: <Swing Dev> Review Request for JDK-8174161

alexander.kouznetsov

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/

 

Best Regards,

Vikrant

 

From: Vikrant Agarwal [[hidden email]]
Sent: Tuesday, February 14, 2017 11:38 AM
To: Sergey Bylokhov
Cc: '[hidden email]'; 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: [hidden email]; 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:

 

 

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