Hi, Petr.
The fix looks good. But before the push, can you split "very long lines" and replace:
OSInfo.getOSType().name().toLowerCase().contains("mac")
to
OSInfo.getOSType() == OSInfo.OSType.MACOSX


On 28.11.2013 16:25, Petr Pchelko wrote:
Hello, AWT Team.

Please review the fix for the issue:
https://bugs.openjdk.java.net/browse/JDK-7152982
The fix is available at:
http://cr.openjdk.java.net/~pchelko/7152982/webrev/

The fix adds AutoDelay to the robot. Also on Mac we are using unicode 
characters to specify the event modifiers. These characters were added to the 
test, so it now pass.

Thank you.
With best regards. Petr.


--
Best regards, Sergey.

Reply via email to