Re: Swing Dev> Review Request JDK-8158205 HiDPI hand cursor broken on Windows

2016-07-08 Thread Semyon Sadetsky
+1 --Semyon On 7/8/2016 10:49 AM, Alexandr Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 7/8/2016 10:31 AM, Rajeev Chamyal wrote: Hello Alexandr, Please review the updated webrev. http://cr.openjdk.java.net/~rchamyal/8158205/webrev.01/

Re: 8160438: [PIT][macosx] [TEST_BUG] javax/swing/plaf/nimbus/8057791/bug8057791.java fails

2016-07-08 Thread Yuri Nesterenko
It pass for me if I provide some time to process native events after the user activity simulation. For instance, setAutoDelay(50) for robot does that plus waitForIdle() after every mouseMove(). In this case, the part with that additional check and a (misleading, I think) comment about the color

Re: Swing Dev> Review Request JDK-8158205 HiDPI hand cursor broken on Windows

2016-07-08 Thread Alexandr Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 7/8/2016 10:31 AM, Rajeev Chamyal wrote: Hello Alexandr, Please review the updated webrev. http://cr.openjdk.java.net/~rchamyal/8158205/webrev.01/ Test was always passing without

Re: Swing Dev> Review Request JDK-8158205 HiDPI hand cursor broken on Windows

2016-07-08 Thread Rajeev Chamyal
Hello Alexandr, Please review the updated webrev. http://cr.openjdk.java.net/~rchamyal/8158205/webrev.01/ Test was always passing without fix also. I have converted it into manual test. Regards, Rajeev Chamyal From: Alexandr Scherbatiy Sent: 05 July 2016 13:29 To: Rajeev Chamyal;