<AWT Dev> Review request for 6384991: TrayIcon ActionListener called at wrong time
ambarish.rapte at oracle.com
Thu Apr 28 16:56:25 UTC 2016
Thanks for the review
I have updated the test case as per the review comments.
Also updated test case with few more scenarios & Os specific conditions.
Please review the updated webrev.01,
From: prasanta sadhukhan
Sent: Tuesday, April 19, 2016 11:25 AM
To: Ambarish Rapte; awt-dev at openjdk.java.net
Cc: Sergey Bylokhov; Semyon Sadetsky
Subject: Re: <AWT Dev> Review request for 6384991: TrayIcon ActionListener called at wrong time
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?
On 4/19/2016 10:51 AM, Ambarish Rapte wrote:
Gentle reminder for review.
From: Ambarish Rapte
Sent: Tuesday, April 12, 2016 10:03 AM
To: Sergey Bylokhov; Semyon Sadetsky; Prasanta Sadhukhan; HYPERLINK "mailto:awt-dev at openjdk.java.net"awt-dev at openjdk.java.net
Subject: <AWT Dev> Review request for 6384991: TrayIcon ActionListener called at wrong time
Please review the fix for JDK9,
On Unix platform, System TrayIcon react on single click.
But Action Listener was called on double click similar to windows.
Changed to generate ActionEvent on single click for unix platform.
Modified existing test to test this fix.
No other regression or JCK test fail due to this fix.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the awt-dev