Re: [9] Review Request for 8145784: https://bugs.openjdk.java.net/browse/JDK-8145784

2016-01-12 Thread Sergey Bylokhov
+1 On 23/12/15 18:47, Alexander Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 12/23/2015 5:48 PM, Semyon Sadetsky wrote: The fix was changed because it is not possible to get individual screen windows under Xinerama correctly. Please, review the updated solution:

Re: [9] Review Request for 8145784: https://bugs.openjdk.java.net/browse/JDK-8145784

2015-12-25 Thread Semyon Sadetsky
On 12/25/2015 7:18 PM, Sergey Bylokhov wrote: Does the getRGBPixels() has the same issue? Is this bug is a regression or it was there from the beginning? No screenshots work. I think it doesn't work since Xinerama support is added. On 23/12/15 17:48, Semyon Sadetsky wrote: The fix was changed

Re: [9] Review Request for 8145784: https://bugs.openjdk.java.net/browse/JDK-8145784

2015-12-25 Thread Sergey Bylokhov
Does the getRGBPixels() has the same issue? Is this bug is a regression or it was there from the beginning? On 23/12/15 17:48, Semyon Sadetsky wrote: The fix was changed because it is not possible to get individual screen windows under Xinerama correctly. Please, review the updated solution:

Re: [9] Review Request for 8145784: https://bugs.openjdk.java.net/browse/JDK-8145784

2015-12-23 Thread Semyon Sadetsky
The fix was changed because it is not possible to get individual screen windows under Xinerama correctly. Please, review the updated solution: http://cr.openjdk.java.net/~ssadetsky/8145784/webrev.01/ The pointer moment coordinates are changed to be relative to the origin of the chosen screen.

Re: [9] Review Request for 8145784: https://bugs.openjdk.java.net/browse/JDK-8145784

2015-12-23 Thread Alexander Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 12/23/2015 5:48 PM, Semyon Sadetsky wrote: The fix was changed because it is not possible to get individual screen windows under Xinerama correctly. Please, review the updated solution:

[9] Review Request for 8145784: https://bugs.openjdk.java.net/browse/JDK-8145784

2015-12-23 Thread Semyon Sadetsky
Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8145784 webrev: http://cr.openjdk.java.net/~ssadetsky/8145784/webrev.00/ Robot used wrong root window for the mouse move because when the Xinerama is on JDK treats multiple screens as a single joint root window,