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

 The array of sizes is only used for the getResolutionVariants() method.
 The images are scaled according to the mapper function.

The current fix creates only one size for the resolution variants list in case
  if they are not passed to the constructor.

  Thanks,
  Alexandr.

On 3/26/2014 4:30 PM, Sergey Bylokhov wrote:
Hello, Alexander.
I think it will not work if the user set scale=1.5?

On 3/26/14 4:18 PM, Alexander Scherbatiy wrote:

  Hello,

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

   - MultiResolution image is used instead of image width/height scaling
   - Bounds are used in the key instead of the scale factor.


   Thanks,
   Alexandr.

On 3/25/2014 8:26 PM, Sergey Bylokhov wrote:
Hello, Alexander.
You cannot skip scalfactor as a key, because images with different scale are different.

On 3/25/14 8:10 PM, Alexander Scherbatiy wrote:

Hello,

Could you review the fix:
  bug: https://bugs.openjdk.java.net/browse/JDK-8032667
  webrev: http://cr.openjdk.java.net/~alexsch/8032667/webrev.00

High resolution image width and height were calculated using only scale factor in the AquaPainter class. The fix calculates image width and height based on the graphics transform.

Thanks,
Alexandr.







Reply via email to