Hi, Konstantin.
The fix looks fine, but please split your new LONG line. before the push.
On 3/21/14 2:44 PM, Konstantin Shefov wrote:
Thanks for the review, look at the fix
http://cr.openjdk.java.net/~kshefov/8022531/webrev.01/
-Konstantin
On 21.03.2014 14:46, Petr Pchelko wrote:
Hello, Konstanting.
After some discussion I've got a comment: what if some platform
actually have an "OK" button and not "Save"?
Could you please update the fix to say something like: "... and click
a confirmation button (OK, Save or any other platform-dependent name)"
With best regards. Petr.
On 21.03.2014, at 14:31, Konstantin Shefov
<konstantin.she...@oracle.com> wrote:
Hello AWT team,
please review the fix
http://cr.openjdk.java.net/~kshefov/8022531/webrev.00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8022531
It is simple manual test fix.
Thanks
-Konstantin
--
Best regards, Sergey.