<AWT Dev> Review request for 6384991: TrayIcon ActionListener called at wrong time

[prasanta sadhukhan]

Hi Ambarish,

Fix looks ok to me.
Regarding test, the copyright should be changed to 2016. Also, the 
initial bugid is missing, since you added bugid can you probably add the 
initial bugid too?
Please change the wildcard imports to specific imports.
Did you test in oel too? The test is skipped for oel if double click is 
done on systray but now you are supporting single-click so I guess we 
should remove the skipping and test in oel, right?

Regards
Prasanta
On 4/19/2016 10:51 AM, Ambarish Rapte wrote:
>
> Hi,
>
> Gentle reminder for review…
>
> Regards,
>
> Ambarish
>
> *From:*Ambarish Rapte
> *Sent:* Tuesday, April 12, 2016 10:03 AM
> *To:* Sergey Bylokhov; Semyon Sadetsky; Prasanta Sadhukhan; 
> awt-dev at openjdk.java.net
> *Subject:* <AWT Dev> Review request for 6384991: TrayIcon 
> ActionListener called at wrong time
>
> Hi All,
>
>                 Please review the fix for JDK9,
>
>                 Webrev: 
> http://cr.openjdk.java.net/~arapte/6384991/webrev.00/ 
> <http://cr.openjdk.java.net/%7Earapte/6384991/webrev.00/>
>
>                 Bug: https://bugs.openjdk.java.net/browse/JDK-6384991
>
> Issue:
>
>                 On Unix platform, System TrayIcon react on single click.
>
>                 But Action Listener was called on double click similar 
> to windows.
>
> Fix:
>
>                 Changed to generate ActionEvent on single click for 
> unix platform.
>
> Verification:
>
>                 Modified existing test to test this fix.
>
>                 No other regression or JCK test fail due to this fix.
>
> Regards,
>
> Ambarish
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20160419/4d2526d4/attachment.html>