41016 2010-06-22 15:39:15 -0700 REGRESSION: keycode for "N" changes when command+option are down 2010-06-22 16:03:35 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Mac All NEW InRadar P1 Normal --- 1 s+webkit webkit-unassigned ap simon.fraser oldest_to_newest 241489 0 59423 s+webkit 2010-06-22 15:39:15 -0700 Created attachment 59423 Example. Warning: eats key presses. In the r57958 nightly (covering r57791:57958), the key code for the N key onkeydown, when the command and option modifiers are down, changes from 78 (expected) to 192 (what?). r57958 is in the wild at this point, in Safari 5 at least. 241492 1 simon.fraser 2010-06-22 15:47:12 -0700 Broke between r57948 and r57951. I guess http://trac.webkit.org/changeset/57951 241495 2 simon.fraser 2010-06-22 15:48:44 -0700 <rdar://problem/8119979> 241502 3 ap 2010-06-22 16:03:35 -0700 That's key code for tilde, the character one can see in Keyboard Viewer at N position with Cmd+Option depressed. It's usual location on Apple U.S. keyboards is to the left of '1' key. Per bug 37776, Cmd-key combos have to use translated character code to produce key codes, and not physical key code. But doing so makes us handle Cmd+Option+N as tilde. Cmd+Option+N should produce 'N' keyCode for U.S. keyboard, same for Dvorak-QWERTY keyboard, but 'B' keyCode for plain Dvorak keyboard. This is tricky, I'm not sure how to implement such a rule. 59423 2010-06-22 15:39:15 -0700 2010-06-22 15:39:15 -0700 Example. Warning: eats key presses. keycodes.html text/html 523 s+webkit PCFET0NUWVBFIGh0bWw+Cgo8aHRtbD4KPGhlYWQ+Cgk8dGl0bGU+V2ViS2l0IHI1Nzc5MTo1Nzk1 OCBOIGtleWNvZGU8L3RpdGxlPgoJPHN0eWxlIHR5cGU9InRleHQvY3NzIiBtZWRpYT0ic2NyZWVu Ij4KCQlkaXZ7CgkJCXRleHQtYWxpZ246IGNlbnRlcjsKCQkJZm9udDogMThweCBIZWx2ZXRpY2Es IEFyaWFsLCBzYW5zLXNlcmlmOwoJCX0KCTwvc3R5bGU+CjwvaGVhZD4KPGJvZHk+Cgk8ZGl2PiZu ZGFzaDsgUHJlc3Mgc29tZSBrZXlzIChlc3BlY2lhbGx5IHRoZSBOIGtleSwgd2l0aCBhbmQgd2l0 aG91dCBjb21tYW5kK29wdGlvbiBoZWxkIGRvd24pISAmbmRhc2g7PC9kaXY+Cgk8c2NyaXB0IHR5 cGU9InRleHQvamF2YXNjcmlwdCI+CgkJdmFyIGRpdiA9IGRvY3VtZW50LmdldEVsZW1lbnRzQnlU YWdOYW1lKCdkaXYnKVswXTsKCQlkb2N1bWVudC5kb2N1bWVudEVsZW1lbnQub25rZXlkb3duID0g ZnVuY3Rpb24oZSl7IGRpdi5pbm5lclRleHQgPSBlLmtleUNvZGU7IHJldHVybiBmYWxzZTsgfTsK CTwvc2NyaXB0Pg==