Hello,

Could you review the updated fix:
  http://cr.openjdk.java.net/~alexsch/8143064/webrev.01

The MultiResolutionImage image is used for the icons painting.

Thanks,
Alexandr.

On 11/16/2015 7:07 PM, Sergey Bylokhov wrote:
Hi, Alexander.
 109         AffineTransform tx = ((Graphics2D) g).getTransform();
110 int sw = tx.isIdentity() ? w : (int) Math.round(w * tx.getScaleX()); 111 int sh = tx.isIdentity() ? h : (int) Math.round(h * tx.getScaleY());

I think that it is not necessary that !isIdentity transform return non-zero value from the getScaleX/Y method. I recall that exactly the same bug on OSX was fixed via MultiResolutionCachedImage, why we cannot do the same here?

On 16.11.15 17:43, Alexander Scherbatiy wrote:
Could you review the fix:
   bug: https://bugs.openjdk.java.net/browse/JDK-8143064
   webrev: http://cr.openjdk.java.net/~alexsch/8143064/webrev.00

   Icon image sizes are scaled in sun.swing.CachedPainter.

   Thanks,
   Alexandr.




Reply via email to