Looks fine.

> 
>  Hello,
> 
> Could you review the updated fix:
>  http://cr.openjdk.java.net/~alexsch/8175301/webrev.01
> 
> There are two places where D3DSurfaceData.swapBuffers() is called 
> D3DScreenUpdateManager.run() and D3DGraphicsConfig.flip(...).
> D3DScreenUpdateManager.run()  uses the surface bounds which are already 
> scaled.
> D3DGraphicsConfig.flip(...) uses coordinates in the user space.
> 
> The fix converts the coordinates in the D3DGraphicsConfig.flip(...) to the 
> device space and converts the coordinates to the user space in the 
> D3DSurfaceData.swapBuffers() method to pass them to the repaint() method.
> 
> Thanks,
> Alexandr.
> 
> On 3/7/2017 4:58 PM, Alexandr Scherbatiy wrote:
>> 
>> Hello,
>> 
>> Could you review the fix:
>>  bug: https://bugs.openjdk.java.net/browse/JDK-8175301
>>  webrev: http://cr.openjdk.java.net/~alexsch/8175301/webrev.00
>> 
>>  D3DSurfaceData.swapBuffers() is updated to scale passed bounds on HiDPI 
>> display.
>> 
>> Thanks,
>> Alexandr.
>> 
> 

Reply via email to