Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(722)

Unified Diff: remoting/tools/javascript_key_tester/keyboard_map.js

Issue 910483002: Remove JS summary. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update version number. Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/tools/javascript_key_tester/keyboard_map.js
diff --git a/remoting/tools/javascript_key_tester/keyboard_map.js b/remoting/tools/javascript_key_tester/keyboard_map.js
index 7ca13665c092e4060d64c636d58783817151fc1a..48c500b3d39ee12cb6f60dda8395aaf01b0af539 100644
--- a/remoting/tools/javascript_key_tester/keyboard_map.js
+++ b/remoting/tools/javascript_key_tester/keyboard_map.js
@@ -23,7 +23,7 @@ var keyboardMap = [
'',
'Shift',
'Control',
- 'Alt',
+ 'Alt', // NB: *not* LeftAlt--not all keyboard layouts make the distinction.
Wez 2015/02/07 02:14:07 I don't understand this comment; what's it got to
'Pause',
'CapsLock',
'Kana',
@@ -230,7 +230,7 @@ var keyboardMap = [
'Quote',
'',
'Meta',
- 'AltGr',
+ 'RightAlt',
Wez 2015/02/07 02:14:07 The 'code' name for this key is AltRight, not Righ
'',
'WIN_ICO_HELP',
'WIN_ICO_00',
« no previous file with comments | « no previous file | remoting/tools/javascript_key_tester/manifest.json » ('j') | remoting/tools/javascript_key_tester/manifest.json » ('J')

Powered by Google App Engine
This is Rietveld 408576698