Hi Semyon,
        Fix looks good to me.
Regards,
Ambarish

-----Original Message-----
From: Semyon Sadetsky 
Sent: Friday, April 14, 2017 3:15 AM
To: awt-dev
Subject: <AWT Dev> [10] Review request for 8132299: Test 
java/awt/Robot/ModifierRobotKey/ModifierRobotKeyTest.java fails on Linux

Hello,

Please review fix for JDK9:

bug: https://bugs.openjdk.java.net/browse/JDK-8132299

webrev: http://cr.openjdk.java.net/~ssadetsky/8132299/webrev.00/

The issue itself is a linux/solaris specific. Alt-Gr modifier bit (1<<5) is not 
set in the KeyPress event which is sent by the closing key in the combination 
(which action should be modified). On platforms other than linux/solaris the 
ModifierRobotKeyTest.java successfully passes but takes a lot of time to 
execute.

Linux has two special keys assigned for 5th column as xmodmap command
report:

mod5        ISO_Level3_Shift (0x5c),  Mode_switch (0xcb)

Java implementation uses Mode_switch native key for mapping to Alt-Gr. 
But Mode_switch is not processed as a modifier key by the native platform while 
ISO_Level3_Shift is processed as modifier. So, changing this mapping solves the 
problem.

Also, the ModifierRobotKeyTest.java is modified in the fix to decrease its 
execution time.

--Semyon

Reply via email to