Generate non-located windowsKeyCode for the WebInputEvents.

ui/events/keycodes/keyboard_code_conversion_android.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/events/keycodes/keyboard_code_conversion_android.h"
 
 #include <android/keycodes.h>
 
 namespace ui {
 
-namespace {
-
-// The Android NDK does not provide values for these yet:
-enum {
-  AKEYCODE_ESCAPE          = 111,
-  AKEYCODE_FORWARD_DEL     = 112,
-  AKEYCODE_CTRL_LEFT       = 113,
-  AKEYCODE_CTRL_RIGHT      = 114,
-  AKEYCODE_CAPS_LOCK       = 115,
-  AKEYCODE_SCROLL_LOCK     = 116,
-  AKEYCODE_META_LEFT       = 117,
-  AKEYCODE_META_RIGHT      = 118,
-  AKEYCODE_BREAK           = 121,
-  AKEYCODE_MOVE_HOME       = 122,
-  AKEYCODE_MOVE_END        = 123,
-  AKEYCODE_INSERT          = 124,
-  AKEYCODE_MEDIA_PLAY      = 126,
-  AKEYCODE_MEDIA_PAUSE     = 127,
-  AKEYCODE_F1              = 131,
-  AKEYCODE_F2              = 132,
-  AKEYCODE_F3              = 133,
-  AKEYCODE_F4              = 134,
-  AKEYCODE_F5              = 135,
-  AKEYCODE_F6              = 136,
-  AKEYCODE_F7              = 137,
-  AKEYCODE_F8              = 138,
-  AKEYCODE_F9              = 139,
-  AKEYCODE_F10             = 140,
-  AKEYCODE_F11             = 141,
-  AKEYCODE_F12             = 142,
-  AKEYCODE_NUM_LOCK        = 143,
-  AKEYCODE_NUMPAD_0        = 144,
-  AKEYCODE_NUMPAD_1        = 145,
-  AKEYCODE_NUMPAD_2        = 146,
-  AKEYCODE_NUMPAD_3        = 147,
-  AKEYCODE_NUMPAD_4        = 148,
-  AKEYCODE_NUMPAD_5        = 149,
-  AKEYCODE_NUMPAD_6        = 150,
-  AKEYCODE_NUMPAD_7        = 151,
-  AKEYCODE_NUMPAD_8        = 152,
-  AKEYCODE_NUMPAD_9        = 153,
-  AKEYCODE_NUMPAD_DIVIDE   = 154,
-  AKEYCODE_NUMPAD_MULTIPLY = 155,
-  AKEYCODE_NUMPAD_SUBTRACT = 156,
-  AKEYCODE_NUMPAD_ADD      = 157,
-  AKEYCODE_NUMPAD_DOT      = 158,
-  AKEYCODE_VOLUME_MUTE     = 164,
-  AKEYCODE_CHANNEL_UP      = 166,
-  AKEYCODE_CHANNEL_DOWN    = 167,
-};
-
-}  // namespace
-
 KeyboardCode KeyboardCodeFromAndroidKeyCode(int keycode) {

[Wez, 2015/09/03 06:05:47]

Couldn't we just replace this function with a keycode->DomCode mapping, and then
use the keycode_conversion routines to translate from that to location,
KeyboardCode, etc, given that this function already assumes a US English layout?

[dtapuska, 2015/09/03 13:39:40]

It makes more sense to me to go from scancode->DomCode (via evdev code); and
keeping keycode->MS VKEY consistent since "keycode" is already mapped via the
android keylayout files. Perhaps I'm wrong?

[dtapuska, 2015/09/03 14:05:49]

Ok I dug a little more and I do think we can avoid this table. See my other
change where I populate the DomCode; once that is available we should be able to
call DomCodeToUsLayoutKeyboardCode and remove this table.

I'll block another patch of this CL to remove the table on that other CL.

[dtapuska, 2015/09/08 17:18:26]

Done.

   // Does not provide all key codes, and does not handle all keys.
   switch (keycode) {
     case AKEYCODE_DEL:
       return VKEY_BACK;
     case AKEYCODE_TAB:
       return VKEY_TAB;
     case AKEYCODE_CLEAR:
       return VKEY_CLEAR;
     case AKEYCODE_DPAD_CENTER:
     case AKEYCODE_ENTER:
       return VKEY_RETURN;
     case AKEYCODE_SHIFT_LEFT:
-      return VKEY_LSHIFT;
+      return VKEY_SHIFT;
     case AKEYCODE_SHIFT_RIGHT:
-      return VKEY_RSHIFT;
+      return VKEY_SHIFT;
     case AKEYCODE_BACK:
       return VKEY_BROWSER_BACK;
     case AKEYCODE_FORWARD:
       return VKEY_BROWSER_FORWARD;
     case AKEYCODE_SPACE:
       return VKEY_SPACE;
     case AKEYCODE_MOVE_HOME:
       return VKEY_HOME;
     case AKEYCODE_DPAD_LEFT:
       return VKEY_LEFT;
@@ skipping 110 matching lines @@
     case AKEYCODE_VOLUME_MUTE:
       return VKEY_VOLUME_MUTE;
     case AKEYCODE_ESCAPE:
       return VKEY_ESCAPE;
     case AKEYCODE_MEDIA_PLAY:
     case AKEYCODE_MEDIA_PLAY_PAUSE:
       return VKEY_MEDIA_PLAY_PAUSE;
     case AKEYCODE_MOVE_END:
       return VKEY_END;
     case AKEYCODE_ALT_LEFT:
-      return VKEY_LMENU;
+      return VKEY_MENU;
     case AKEYCODE_ALT_RIGHT:
-      return VKEY_RMENU;
+      return VKEY_MENU;
     case AKEYCODE_GRAVE:
       return VKEY_OEM_3;
     case AKEYCODE_APOSTROPHE:
       return VKEY_OEM_3;
     case AKEYCODE_MEDIA_REWIND:
       return VKEY_OEM_103;
     case AKEYCODE_MEDIA_FAST_FORWARD:
       return VKEY_OEM_104;
     case AKEYCODE_PAGE_UP:
       return VKEY_PRIOR;
     case AKEYCODE_PAGE_DOWN:
       return VKEY_NEXT;
     case AKEYCODE_FORWARD_DEL:
       return VKEY_DELETE;
     case AKEYCODE_CTRL_LEFT:
-      return VKEY_LCONTROL;
+      return VKEY_CONTROL;
     case AKEYCODE_CTRL_RIGHT:
-      return VKEY_RCONTROL;
+      return VKEY_CONTROL;
     case AKEYCODE_CAPS_LOCK:
       return VKEY_CAPITAL;
     case AKEYCODE_SCROLL_LOCK:
       return VKEY_SCROLL;
     case AKEYCODE_META_LEFT:
       return VKEY_LWIN;
     case AKEYCODE_META_RIGHT:
-      return VKEY_RWIN;
+      return VKEY_LWIN;
     case AKEYCODE_BREAK:
       return VKEY_PAUSE;
     case AKEYCODE_INSERT:
       return VKEY_INSERT;
     case AKEYCODE_F1:
       return VKEY_F1;
     case AKEYCODE_F2:
       return VKEY_F2;
     case AKEYCODE_F3:
       return VKEY_F3;
@@ skipping 50 matching lines @@
     case AKEYCODE_CHANNEL_UP:
       return VKEY_PRIOR;
     case AKEYCODE_CHANNEL_DOWN:
       return VKEY_NEXT;
     default:
       return VKEY_UNKNOWN;
   }
 }
 
 }  // namespace ui