Hi, Manajit.
I have only a few comments about a style:
- I think it would be good to have two methods createFromImage() the
old(without Observer) and new(with Observer). The old method should pass
null to the new method. In this case CMenuItem/CCustomCursor will not be
changed.
- The parentheses around target are not necessary
CTrayIcon.java:359 if (image != (target).getImage())
On 03/05/2018 04:11, Manajit Halder wrote:
Hi Sergey,
Please review the updated webrev.
http://cr.openjdk.java.net/~mhalder/8029250/webrev.01/
The modified test case moved to closed test as it contains images with
unknown source.
Regards,
Manajit
On 27-Apr-2018, at 4:06 PM, Manajit Halder <manajit.hal...@oracle.com
<mailto:manajit.hal...@oracle.com>> wrote:
Hi All,
Kindly review the following AWT enhancement changes:
Bug: https://bugs.openjdk.java.net/browse/JDK-8029250
Webrev: http://cr.openjdk.java.net/~mhalder/8029250/webrev.00/
Fix:
Added support for gif images (image animation) for Mac system tray.
Before fix only single frame was passed to Mac OS system tray on mouse
click from the Java side.
After fix all the frames are passed at the time interval set in the
image one by one to the Mac OS side.
Note:
The test was moved from closed test to open test along with 3 images:
ball.gif, spot.gif and duke.gif. The test code was rewritten dropping
the applet code used earlier.
Regards,
Manajit
--
Best regards, Sergey.