The test instruction:

> Please check if both of them have correct size and
> the same colouring (white rectagle in a blue mount). In this case please push "Pass".

It is unclear how does it test the fix?
Correct/incorrect size criteria should be revealed better.
However, it is hard to count pixels on the screen by eye. I propose to make the criteria more obvious to testers by selection of the image dimensions and content.

On 3/15/2016 3:54 AM, Sergey Bylokhov wrote:
Hello,
Please review the small fix for jdk9.

According to specification if imageAutoSize property was set to true the image should be the stretched or shrunk to fit the tray icon space. But on OSX this flag is ignored in the native code.

In the fix this flag is taken into account, if it was set to true. otherwise the previous logic is executed:
- Small images are placed as is.
- Big images are proportionally down-scaled(I suppose this is done for some usability issue)

Bug: https://bugs.openjdk.java.net/browse/JDK-8151773
Webrev can be found at: http://cr.openjdk.java.net/~serb/8151773/webrev.00


Reply via email to