Re: Review request for JDK-8038631: Create wrapper for awt.Robot with additional functionality

[Alexander Scherbatiy]

On 4/7/2014 3:56 PM, Dmitriy Ermashov wrote:
Hi, Sergey
Do you have any additions?
http://cr.openjdk.java.net/~yan/8038631/webrev.06/


     Will the AccessController.doPrivileged() block work under the 
Security manager if the ExtendedRobot package is not listed in the 
"package.access" list?

   Thanks,
   Alexandr.

--
Thanks,
Dima


On 04/07/2014 12:32 PM, Alexander Zvegintsev wrote:
Hello Dmitriy,

this version looks good to me.

Thanks,

Alexander.

On 04/04/2014 11:32 PM, Petr Pchelko wrote:
Hello, Dmitry.

This version looks great.

With best regards. Petr.

04 апр. 2014 г., в 8:16 после полудня, Dmitriy Ermashov 
<dmitriy.ermas...@oracle.com> написал(а):

Hi,
Please, review the changeset for:
https://bugs.openjdk.java.net/browse/JDK-8038631

Webrev is here:
http://cr.openjdk.java.net/~yan/8038631/webrev.06/

Last changes consist of:
1. Removed if clause from class initialization section

—

Thanks,
Dima