Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-03-01 Thread Sergey Bylokhov
On 01/03/2018 02:41, Shashidhara Veerabhadraiah wrote: Sergey, we have a floating point dpi scale(dpi value/96) and we use it to scale down and up to convert from user space to device space and vice versa. This code is present at awt_Win32GraphicsDevice.cpp. In this code if you see we use

Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-03-01 Thread Shashidhara Veerabhadraiah
nd regards, Shashi -Original Message- From: Sergey Bylokhov Sent: Thursday, March 1, 2018 2:23 AM To: Shashidhara Veerabhadraiah <shashidhara.veerabhadra...@oracle.com>; Krishna Addepalli <krishna.addepa...@oracle.com>; awt-dev@openjdk.java.net Subject: Re: [11] JDK-81

Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-02-28 Thread Sergey Bylokhov
On 27/02/2018 07:51, Shashidhara Veerabhadraiah wrote: Threshold is added because I think it is quite possible especially with hidpi(fractional) since convert back and forth from user space to device coordinates which typically results in plus or minus 1 pixel correction(because of division

Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-02-27 Thread Krishna Addepalli
) From: Shashidhara Veerabhadraiah <shashidhara.veerabhadra...@oracle.com> To: Philip Race <philip.r...@oracle.com> Cc: awt-dev@openjdk.java.net Subject: Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails Message-ID: &l

Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-02-26 Thread Phil Race
e.com>; awt-dev@openjdk.java.net > Subject: Re: [11] JDK-8196017: > java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails > > What Windows 10 version are you running ? As discussed a few times ... > probably off-list .. > there is a bug in Windows 10 Fall Creato

Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-02-26 Thread Shashidhara Veerabhadraiah
gt; Cc: Sergey Bylokhov <sergey.bylok...@oracle.com>; awt-dev@openjdk.java.net Subject: Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails What Windows 10 version are you running ? As discussed a few times ... probably off-list .. there is a bug in Win

Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-02-26 Thread Philip Race
Veerabhadraiah<shashidhara.veerabhadra...@oracle.com>; awt-dev@openjdk.java.net Subject: Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails Hi, Shashi. Why this test should be platform specific? It shows two common frames and use some gener

Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-02-26 Thread Shashidhara Veerabhadraiah
Sent: Tuesday, February 27, 2018 4:05 AM To: Shashidhara Veerabhadraiah <shashidhara.veerabhadra...@oracle.com>; awt-dev@openjdk.java.net Subject: Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails Hi, Shashi. Why this test should be platform sp

Re: [11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-02-26 Thread Sergey Bylokhov
Hi, Shashi. Why this test should be platform specific? It shows two common frames and use some generic api for mouse events. This test also was updated in JDK-8027154 and I assume it should works on macOS after that, maybe we have a new regression here? On 24/02/2018 07:32, Shashidhara

[11] JDK-8196017: java/awt/Mouse/GetMousePositionTest/GetMousePositionWithPopup.java fails

2018-02-24 Thread Shashidhara Veerabhadraiah
Hi All, Please review a test fix for the below bug: Bug: https://bugs.openjdk.java.net/browse/JDK-8196017 Webrev: http://cr.openjdk.java.net/~sveerabhadra/8196017/webrev.00/ Summary: The original bug https://bugs.openjdk.java.net/browse/JDK-8012026 fixed the code only