--- old/src/java.desktop/macosx/native/libawt_lwawt/awt/AWTEvent.m 2016-05-14 01:59:27.000000000 +0530 +++ new/src/java.desktop/macosx/native/libawt_lwawt/awt/AWTEvent.m 2016-05-14 01:59:27.000000000 +0530 @@ -131,7 +131,7 @@ {0x3A, NO, KL_UNKNOWN, java_awt_event_KeyEvent_VK_ALT}, // **** {0x3B, NO, KL_UNKNOWN, java_awt_event_KeyEvent_VK_CONTROL}, // **** {0x3C, NO, KL_UNKNOWN, java_awt_event_KeyEvent_VK_UNDEFINED}, - {0x3D, NO, KL_UNKNOWN, java_awt_event_KeyEvent_VK_UNDEFINED}, + {0x3D, NO, KL_UNKNOWN, java_awt_event_KeyEvent_VK_ALT_GRAPH}, {0x3E, NO, KL_UNKNOWN, java_awt_event_KeyEvent_VK_UNDEFINED}, {0x3F, NO, KL_UNKNOWN, java_awt_event_KeyEvent_VK_UNDEFINED}, // the 'fn' key on PowerBooks {0x40, NO, KL_STANDARD, java_awt_event_KeyEvent_VK_F17}, @@ -293,7 +293,7 @@ 61, java_awt_event_InputEvent_ALT_DOWN_MASK | java_awt_event_InputEvent_ALT_GRAPH_DOWN_MASK, java_awt_event_InputEvent_ALT_MASK | java_awt_event_InputEvent_ALT_GRAPH_MASK, - java_awt_event_KeyEvent_VK_ALT + java_awt_event_KeyEvent_VK_ALT | java_awt_event_KeyEvent_VK_ALT_GRAPH }, { NSCommandKeyMask, --- old/src/java.desktop/macosx/native/libawt_lwawt/awt/CRobot.m 2016-05-14 01:59:28.000000000 +0530 +++ new/src/java.desktop/macosx/native/libawt_lwawt/awt/CRobot.m 2016-05-14 01:59:28.000000000 +0530 @@ -33,6 +33,7 @@ #import "LWCToolkit.h" #import "sun_lwawt_macosx_CRobot.h" #import "java_awt_event_InputEvent.h" +#import "java_awt_event_KeyEvent.h" #import "sizecalc.h" // Starting number for event numbers generated by Robot. @@ -265,19 +266,36 @@ * events. So, I've ended up disabling it and opted for another approach * that uses Accessibility API instead. */ + CGKeyCode keyCode = GetCGKeyCode(javaKeyCode); - AXUIElementRef elem = AXUIElementCreateSystemWide(); - AXUIElementPostKeyboardEvent(elem, (CGCharCode)0, keyCode, keyPressed); - CFRelease(elem); + /* + * JDK-8155740: AXUIElementPostKeyboardEvent posts 0 key code for all + * the modifier keys with key codes (16, 17,18, 20, 157) and also for + * newly added modifier key VK_ALT_GRAPH. But it posts correct key code + * for all the other keys. On the other hand CGEventCreateKeyboardEvent + * posts correct key code for all the keys except numeric keys (0 to 9). + * CGEventCreateKeyboardEvent posts wrong key codes for the number keys + * 0 to 9. Instead of posting number key codes it posts Numpad key codes + * for the corresponding number key. For example Numpad0 key is posted for + * number 0 and simillarly for remaining num keys. Based on this behaviour + * AXUIElementPostKeyboardEvent is used to post only numeric key events + * and CGEventCreateKeyboardEvent used for posting all other key events. + */ -#if 0 - CGEventRef event = CGEventCreateKeyboardEvent(NULL, keyCode, keyPressed); - if (event != NULL) { - CGEventPost(kCGSessionEventTap, event); - CFRelease(event); + if ((javaKeyCode >= java_awt_event_KeyEvent_VK_0) && + (javaKeyCode <= java_awt_event_KeyEvent_VK_9)) + { + AXUIElementRef elem = AXUIElementCreateSystemWide(); + AXUIElementPostKeyboardEvent(elem, (CGCharCode)0, keyCode, keyPressed); + CFRelease(elem); + } else { + CGEventRef event = CGEventCreateKeyboardEvent(NULL, keyCode, keyPressed); + if (event != NULL) { + CGEventPost(kCGSessionEventTap, event); + CFRelease(event); + } } -#endif } /* --- old/src/java.desktop/macosx/native/libawt_lwawt/awt/CRobotKeyCode.m 2016-05-14 01:59:29.000000000 +0530 +++ new/src/java.desktop/macosx/native/libawt_lwawt/awt/CRobotKeyCode.m 2016-05-14 01:59:29.000000000 +0530 @@ -53,6 +53,7 @@ [NSNumber numberWithInt : OSX_Shift], [NSNumber numberWithInt : java_awt_event_KeyEvent_VK_SHIFT], [NSNumber numberWithInt : OSX_Control], [NSNumber numberWithInt : java_awt_event_KeyEvent_VK_CONTROL], [NSNumber numberWithInt : OSX_Option], [NSNumber numberWithInt : java_awt_event_KeyEvent_VK_ALT], + [NSNumber numberWithInt : OSX_RightOption], [NSNumber numberWithInt : java_awt_event_KeyEvent_VK_ALT_GRAPH], [NSNumber numberWithInt : OSX_CapsLock], [NSNumber numberWithInt : java_awt_event_KeyEvent_VK_CAPS_LOCK], [NSNumber numberWithInt : OSX_Escape], [NSNumber numberWithInt : java_awt_event_KeyEvent_VK_ESCAPE], [NSNumber numberWithInt : OSX_kVK_Space], [NSNumber numberWithInt : java_awt_event_KeyEvent_VK_SPACE], --- old/test/java/awt/keyboard/AllKeyCode/AllKeyCode.java 2016-05-14 01:59:31.000000000 +0530 +++ new/test/java/awt/keyboard/AllKeyCode/AllKeyCode.java 2016-05-14 01:59:30.000000000 +0530 @@ -22,15 +22,14 @@ */ /* - @test - @bug 8149456 8147834 8150230 - @requires os.family == "mac" - @summary KeyEvents for all keys - @run main AllKeyCode -*/ + @test + @bug 8149456 8147834 8150230 8155740 + @requires os.family == "mac" + @summary Tests key codes for all keys supported in Java for Mac OS X. + @run main AllKeyCode + */ import java.awt.AWTException; -import java.awt.GridBagLayout; import java.awt.Robot; import java.awt.event.KeyEvent; import java.awt.event.KeyListener; @@ -149,7 +148,8 @@ KeyEvent.VK_F17, KeyEvent.VK_F18, KeyEvent.VK_F19, - KeyEvent.VK_F20 + KeyEvent.VK_F20, + KeyEvent.VK_ALT_GRAPH }; keyPressedIndex = -1;