Re: [OpenJDK 2D-Dev] [9] Review request for 8151303 [macosx] [hidpi] JButton's low-res. icon is visible when clicking on it

2016-08-15 Thread Jim Graham
Looks good. +1 ...jim On 08/15/2016 08:47 AM, Alexander Scherbatiy wrote: Could you review the updated fix: http://cr.openjdk.java.net/~alexsch/8151303/webrev.04 The MRCI.sizes arrays is reused for for MultiResolutionCachedImage. Thanks, Alexandr. On 11/08/16

Re: [OpenJDK 2D-Dev] [9] Review request for 8151303 [macosx] [hidpi] JButton's low-res. icon is visible when clicking on it

2016-08-15 Thread Alexander Scherbatiy
Could you review the updated fix: http://cr.openjdk.java.net/~alexsch/8151303/webrev.04 The MRCI.sizes arrays is reused for for MultiResolutionCachedImage. Thanks, Alexandr. On 11/08/16 23:10, Jim Graham wrote: Hi Alexandr, Should something be done to transfer the array of sizes to the

[9] Review request for 8164032 JViewport backing store image is not scaled on HiDPI display

2016-08-15 Thread Alexandr Scherbatiy
Hello, Could you review the fix: bug: https://bugs.openjdk.java.net/browse/JDK-8164032 webrev: http://cr.openjdk.java.net/~alexsch/8164032/webrev.00 The fix scales the JViewport backing store image when graphics transform is not identity. Thanks, Alexandr.

Re: [9] Review request for 8156217 Selected text is shifted on HiDPI display

2016-08-15 Thread Alexandr Scherbatiy
Hello, Could you review the updated fix? webrev which contains only change in public API: http://cr.openjdk.java.net/~alexsch/8156217/webrev.05/public-api webrev with contains all changes: http://cr.openjdk.java.net/~alexsch/8156217/webrev.05/all - methods with int coordinates

Review Request for 8163261: regression on Linux: java/awt/LightweightDispatcher/LWDispatcherMemoryLeakTest.java

2016-08-15 Thread Ambarish Rapte
Hi, Please review fix for JDK9, Bug: https://bugs.openjdk.java.net/browse/JDK-8163261 Webrev: http://cr.openjdk.java.net/~arapte/8163261/webrev.00/ Issue: Reference to JButton was not getting collected by GC. Cause: