Located vs non-located ui::KeyboardCode functions.

ui/events/keycodes/keyboard_code_conversion.cc

 // Copyright (c) 2011 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.h"
 
 #include "ui/events/event_constants.h"
+#include "ui/events/keycodes/dom3/dom_code.h"
 #include "ui/events/keycodes/dom3/dom_key.h"
 
 namespace ui {
 
 namespace {
 
 // This table maps a subset of |KeyboardCode| (VKEYs) to DomKey and character.
 // Only values not otherwise handled by GetMeaningFromKeyCode() are here.
 const struct KeyboardCodeToMeaning {
   KeyboardCode key_code;
@@ skipping 99 matching lines @@
 #if defined(OS_POSIX)
     {VKEY_POWER, DomKey::POWER, 0, 0},
     {VKEY_BRIGHTNESS_DOWN, DomKey::BRIGHTNESS_DOWN, 0, 0},
     {VKEY_BRIGHTNESS_UP, DomKey::BRIGHTNESS_UP, 0, 0},
     {VKEY_COMPOSE, DomKey::COMPOSE, 0, 0},
     {VKEY_OEM_103, DomKey::MEDIA_REWIND, 0, 0},
     {VKEY_OEM_104, DomKey::MEDIA_FAST_FORWARD, 0, 0},
 #endif
 };
 
+bool IsRightSideDomCode(DomCode code) {

[Wez, 2015/01/10 02:06:28]

Why are you treating the modifiers to an IsRightSideDomCode() function while the
numeric pad keys are done in-line below?

[kpschoedel, 2015/01/12 16:34:06]

IsRightSideDomCode() exists because |code| doesn't change with remapping or
layout, so we can have for instance DomCode::ALT_LEFT with VKEY_LCONTROL. We
don't have any layouts that move numpad digits (although I can imagine a
'telephone' option). I'll add a IsNumpadDomCode() or equivalent for that if you
like, but see the last comment below.

[Wez, 2015/01/13 03:08:36]

Acknowledged.

+  return (code == DomCode::SHIFT_RIGHT) || (code == DomCode::CONTROL_RIGHT) ||
+         (code == DomCode::ALT_RIGHT) || (code == DomCode::OS_RIGHT);
+}
+
 }  // anonymous namespace
 
 base::char16 GetCharacterFromKeyCode(KeyboardCode key_code, int flags) {
   ui::DomKey dom_key;
   base::char16 character;
   if (GetMeaningFromKeyCode(key_code, flags, &dom_key, &character))
     return character;
   return 0;
 }
 
@@ skipping 117 matching lines @@
       *character = (shift && p->shift_character) ? p->shift_character
                                                  : p->plain_character;
       return true;
     }
   }
   *dom_key = DomKey::UNIDENTIFIED;
   *character = 0;
   return false;
 }
 
+// Determine the non-located VKEY corresponding to a located VKEY.

[kpschoedel, 2015/01/09 20:25:40]

This functionality, used by Ozone keyboard layout, also exists in
DevToolsEventForwarder::VirtualKeyCodeWithoutLocation() and
content::GetWindowsKeyCodeWithoutLocation(), which I will consolidate in a
follow-up CL.

[Wez, 2015/01/13 03:08:36]

Acknowledged.

+KeyboardCode LocatedToNonLocatedKeyboardCode(KeyboardCode key_code) {

[Wez, 2015/01/10 02:06:28]

nit: Could/should this method output the Location as well as the non-located
KeyboardCode?

[Wez, 2015/01/10 02:06:28]

Since you handle numeric pad keys in the reverse mapping, handle them here as
well?

[kpschoedel, 2015/01/12 16:34:06]

There is not currently a ui::Event-level analogue of Blink's DOM_KEY_LOCATION_….
I wouldn't mind adding one.

[kpschoedel, 2015/01/12 16:34:06]

I'll do that in a follow-up CL, since it's a behaviour change from the existing
code, so I'd like to keep it separate from refactoring with no intentional
behaviour change.

[Wez, 2015/01/13 03:08:36]

Arguably not worth it unless we actually need it. :)

If ui::Event ends up containing |key| and |code| then we could have an accessor
that just calls on to IsRightSideDomCode() to synthesize location information.

[Wez, 2015/01/13 03:08:36]

Acknowledged.

[kpschoedel, 2015/01/13 16:21:08]

This is mostly implemented and should trickle in through M42.  

+  switch (key_code) {
+    case VKEY_RWIN:
+      return VKEY_LWIN;
+    case VKEY_LSHIFT:
+    case VKEY_RSHIFT:
+      return VKEY_SHIFT;
+    case VKEY_LCONTROL:
+    case VKEY_RCONTROL:
+      return VKEY_CONTROL;
+    case VKEY_LMENU:
+    case VKEY_RMENU:
+      return VKEY_MENU;
+    default:
+      return key_code;
+  }
+}
+
+// Determine the located VKEY corresponding to a non-located VKEY.
+KeyboardCode NonLocatedToLocatedKeyboardCode(KeyboardCode key_code,
+                                             DomCode dom_code) {
+  switch (key_code) {
+    case VKEY_SHIFT:
+      return IsRightSideDomCode(dom_code) ? VKEY_RSHIFT : VKEY_LSHIFT;
+    case VKEY_CONTROL:
+      return IsRightSideDomCode(dom_code) ? VKEY_RCONTROL : VKEY_LCONTROL;
+    case VKEY_MENU:
+      return IsRightSideDomCode(dom_code) ? VKEY_RMENU : VKEY_LMENU;
+    case VKEY_LWIN:
+      return IsRightSideDomCode(dom_code) ? VKEY_RWIN : VKEY_LWIN;
+    case VKEY_0:
+      return (dom_code == DomCode::NUMPAD0) ? VKEY_NUMPAD0 : VKEY_0;
+    case VKEY_1:
+      return (dom_code == DomCode::NUMPAD1) ? VKEY_NUMPAD1 : VKEY_1;
+    case VKEY_2:
+      return (dom_code == DomCode::NUMPAD2) ? VKEY_NUMPAD2 : VKEY_2;
+    case VKEY_3:
+      return (dom_code == DomCode::NUMPAD3) ? VKEY_NUMPAD3 : VKEY_3;
+    case VKEY_4:
+      return (dom_code == DomCode::NUMPAD4) ? VKEY_NUMPAD4 : VKEY_4;
+    case VKEY_5:
+      return (dom_code == DomCode::NUMPAD5) ? VKEY_NUMPAD5 : VKEY_5;
+    case VKEY_6:
+      return (dom_code == DomCode::NUMPAD6) ? VKEY_NUMPAD6 : VKEY_6;
+    case VKEY_7:
+      return (dom_code == DomCode::NUMPAD7) ? VKEY_NUMPAD7 : VKEY_7;
+    case VKEY_8:
+      return (dom_code == DomCode::NUMPAD8) ? VKEY_NUMPAD8 : VKEY_8;
+    case VKEY_9:
+      return (dom_code == DomCode::NUMPAD9) ? VKEY_NUMPAD9 : VKEY_9;
+    default:
+      return key_code;
+  }
+}
+
 }  // namespace ui