<AWT Dev> <Awt Dev>  Review Request for 8081457: TrayIcon tests fail in OEL 7 only
alexander.zvegintsev at oracle.com
Wed Oct 28 13:05:22 UTC 2015
The fix looks good to me.
On 10/28/2015 03:01 PM, Semyon Sadetsky wrote:
> 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.
>> On 10/13/2015 02:24 PM, Semyon Sadetsky wrote:
>>> 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.
More information about the awt-dev