Hello, Alexander. Could you please add the @Override to the paintIcon method.
Also I suggest to reuse the paintIcon in the createIcon. With best regards. Petr. On 02.04.2014, at 16:20, Alexander Scherbatiy <alexandr.scherba...@oracle.com> wrote: > > Hello, > > Could you review the updated fix: > http://cr.openjdk.java.net/~alexsch/8038113/webrev.01/ > > The CachableJRSUIIcon creates an image twice: in createImage() and in the > AquaPainter.paint() methods. > > The fix paints the icon directly from the AquaPainter.paint() method which > uses the properly scaled image. > > Thanks, > Alexandr. > > On 3/26/2014 6:15 PM, Alexander Scherbatiy wrote: >> >> Hello, >> >> Could you review the fix: >> bug: https://bugs.openjdk.java.net/browse/JDK-8038113 >> webrev: http://cr.openjdk.java.net/~alexsch/8038113/webrev.00 >> >> MultiResolution image is used to create resolution variants for the JTree >> icons. >> >> The fix assumes that AquaPainter uses the graphics transform for the image >> size calculation >> an so depends on the fix for issue 8032667 [macosx] Components cannot be >> rendered in >> http://mail.openjdk.java.net/pipermail/awt-dev/2014-March/007370.html >> >> Thanks, >> Alexandr. >> >
