<AWT Dev> <Awt Dev> [9] Review Request for 8081457: TrayIcon tests fail in OEL 7 only

Semyon Sadetsky semyon.sadetsky at oracle.com
Wed Oct 28 12:01:17 UTC 2015

On 10/14/2015 3:50 PM, Alexander Zvegintsev wrote:
> Hi Semyon,
> As I can see there are a lot similar checks in tests for OEL7, ideally 
> it should be checked against  gnome-shell(probably from 
> DESKTOP_SESSION env variable, or check for running gnome-shell 
> process), but I am OK with the current check. Could you please extract 
> this check to some helper method? It would be helpful, if we will 
> change this check in future.
I have introduced isOel() method for the check : 
gnome-shell is a common session name for both gnome2 and gnome3. So I've 
left the same check.
> Thanks,
> Alexander.
> On 10/13/2015 02:24 PM, Semyon Sadetsky wrote:
>> Hello,
>> Please review fix for JDK9:
>> bug: https://bugs.openjdk.java.net/browse/JDK-8081457
>> webrev: http://cr.openjdk.java.net/~ssadetsky/8081457/webrev.00/
>> In OEL7 the system tray was changed a lot. It is totally hidden by 
>> default and its tray icons only capable to receive click events. 
>> Mouse motion events and double click is not supported in the tray.
>> Also input events are sent to the embedded frame instead of the 
>> canvas with the icon. The mouse press event is sent to the embedded 
>> frame only after button is released.
>> In the proposed fix the mouse click listener is added to the embedded 
>> frame. The tray icon test suite is modified to detect and pass OEL7 
>> system tray.
>> --Semyon

More information about the awt-dev mailing list