<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hello, Alexander.<div><br></div><div>The fix looks good to me too.</div><div><div>
<br class="Apple-interchange-newline"><span style="color: rgb(0, 0, 0); font-family: Helvetica;  font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; display: inline !important; float: none; ">With best regards. Petr.</span><br style="color: rgb(0, 0, 0); font-family: Helvetica;  font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><br style="color: rgb(0, 0, 0); font-family: Helvetica;  font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; ">
</div>
<br><div><div>On 26.03.2014, at 17:54, Sergey Bylokhov <<a href="mailto:Sergey.Bylokhov@oracle.com">Sergey.Bylokhov@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hi, Alexander.<br>The fix looks fine to me. Probably the test can be automated? before the fix COMP->imagex1->imagex2 and the COMP->imagex2 should be the same, and after the fix it should be different?<br><br>On 3/26/14 5:43 PM, Alexander Scherbatiy wrote:<br><blockquote type="cite"><br>  Could you review the updated fix:<br>     <a href="http://cr.openjdk.java.net/~alexsch/8032667/webrev.02">http://cr.openjdk.java.net/~alexsch/8032667/webrev.02</a><br><br> The array of sizes is only used for the getResolutionVariants() method.<br> The images are scaled according to the mapper function.<br><br>  The current fix creates only one size for the resolution variants list in case<br>  if they are not passed to the constructor.<br><br>  Thanks,<br>  Alexandr.<br><br>On 3/26/2014 4:30 PM, Sergey Bylokhov wrote:<br><blockquote type="cite">Hello, Alexander.<br>I think it will not work if the user set scale=1.5?<br><br>On 3/26/14 4:18 PM, Alexander Scherbatiy wrote:<br><blockquote type="cite"><br>  Hello,<br><br>  Could you review the updated fix:<br>     <a href="http://cr.openjdk.java.net/~alexsch/8032667/webrev.01/">http://cr.openjdk.java.net/~alexsch/8032667/webrev.01/</a><br><br>   - MultiResolution image is used instead of image width/height scaling<br>   - Bounds are used in the key instead of the scale factor.<br><br><br>   Thanks,<br>   Alexandr.<br><br>On 3/25/2014 8:26 PM, Sergey Bylokhov wrote:<br><blockquote type="cite">Hello, Alexander.<br>You cannot skip scalfactor as a key, because images with different scale are different.<br><br>On 3/25/14 8:10 PM, Alexander Scherbatiy wrote:<br><blockquote type="cite"><br>Hello,<br><br>Could you review the fix:<br>  bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8032667">https://bugs.openjdk.java.net/browse/JDK-8032667</a><br>  webrev: <a href="http://cr.openjdk.java.net/~alexsch/8032667/webrev.00">http://cr.openjdk.java.net/~alexsch/8032667/webrev.00</a><br><br>  High resolution image width and height were calculated using only scale factor in the AquaPainter class.<br>  The fix calculates image width and height based on the graphics transform.<br><br>Thanks,<br>Alexandr.<br><br></blockquote><br><br></blockquote><br></blockquote><br><br></blockquote><br></blockquote><br><br>-- <br>Best regards, Sergey.<br><br></blockquote></div><br></div></body></html>