<AWT Dev> [9] Review request for 8132123 MultiResolutionCachedImage unnecessarily creates base image to get its size

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Fri Jul 24 11:10:58 UTC 2015


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

  There is one more place where AquaSplitPaneDividerUI requests scaled 
smooth image. It is better to allow the native system to scale the image.
  So I updated the getScaledInstance(width, height, hints) method in the 
MultiResolutionCachedImage. It also allows to avoid base image creation.

On 7/22/2015 8:23 PM, Sergey Bylokhov wrote:
> Hi, Alexander.
> Why you did update of bugid in the test, but did not update the test 
> itself? Did the test fail before the fix?

      There are limited number of loaded native images and they are 
immediately cached so it is hard to write a test which calculates the 
excessive memory usage using only public API.

    I added the new test that uses the internal 
MultiResolutionCachedImage class directly.

   Thanks,
   Alexandr.
>
> On 22.07.15 16:44, Alexander Scherbatiy wrote:
>>
>> Hello,
>>
>> Could you review the fix:
>>   bug: https://bugs.openjdk.java.net/browse/JDK-8132123
>>   webrev: http://cr.openjdk.java.net/~alexsch/8132123/webrev.00
>>
>>   The baseImageWidth/Height and Image.UndefinedProperty are returned 
>> to not trigger base image creation in the MultiResolutionCachedImage.
>>
>>
>> Thanks,
>> Alexandr.
>>
>
>



More information about the awt-dev mailing list