Re: [9] Review Request: 8047226 closed/java/awt/Component/GetScreenLocTest/GetScreenLocTest.html clicks on Unity's tool bar

2015-08-26 Thread Sergey Bylokhov
On 26.08.2015 16:17, Alexander Scherbatiy wrote: On 8/25/2015 6:28 PM, Sergey Bylokhov wrote: On 25.08.15 17:55, Alexander Zvegintsev wrote: I think that there is no need in RuntimeException in mousePressed() callback, it could be replaced with println and break. makes sense, the new version:

Re: [9] Review Request: 8047226 closed/java/awt/Component/GetScreenLocTest/GetScreenLocTest.html clicks on Unity's tool bar

2015-08-26 Thread Alexander Scherbatiy
On 8/25/2015 6:28 PM, Sergey Bylokhov wrote: On 25.08.15 17:55, Alexander Zvegintsev wrote: I think that there is no need in RuntimeException in mousePressed() callback, it could be replaced with println and break. makes sense, the new version: http://cr.openjdk.java.net/~serb/8047226/webrev.0

Re: [9] Review Request: 8047226 closed/java/awt/Component/GetScreenLocTest/GetScreenLocTest.html clicks on Unity's tool bar

2015-08-26 Thread Alexander Zvegintsev
Looks fine. Thanks, Alexander. On 08/25/2015 06:28 PM, Sergey Bylokhov wrote: On 25.08.15 17:55, Alexander Zvegintsev wrote: I think that there is no need in RuntimeException in mousePressed() callback, it could be replaced with println and break. makes sense, the new version: http://cr.ope

Re: [9] Review Request: 8047226 closed/java/awt/Component/GetScreenLocTest/GetScreenLocTest.html clicks on Unity's tool bar

2015-08-25 Thread Sergey Bylokhov
On 25.08.15 17:55, Alexander Zvegintsev wrote: I think that there is no need in RuntimeException in mousePressed() callback, it could be replaced with println and break. makes sense, the new version: http://cr.openjdk.java.net/~serb/8047226/webrev.01 Thanks, Alexander. On 08/25/2015 05:36

Re: [9] Review Request: 8047226 closed/java/awt/Component/GetScreenLocTest/GetScreenLocTest.html clicks on Unity's tool bar

2015-08-25 Thread Alexander Zvegintsev
Hello Sergey, I think that there is no need in RuntimeException in mousePressed() callback, it could be replaced with println and break. Thanks, Alexander. On 08/25/2015 05:36 PM, Sergey Bylokhov wrote: Hello. Please review the fix for jdk9. Test was fixed(the windows were moved to the cen

[9] Review Request: 8047226 closed/java/awt/Component/GetScreenLocTest/GetScreenLocTest.html clicks on Unity's tool bar

2015-08-25 Thread Sergey Bylokhov
Hello. Please review the fix for jdk9. Test was fixed(the windows were moved to the center of the screen), cleaned and was moved to the open. The diff to the previous version is provided [1]. Bug: https://bugs.openjdk.java.net/browse/JDK-8047226 Webrev can be found at: http://cr.openjdk.java.n