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

Unified Diff: ui/events/keycodes/keyboard_code_conversion.cc

Issue 841263005: Use DOM- rather than Windows-based key code for non-layout cases. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address review comments (sadrul) Created 5 years, 8 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: ui/events/keycodes/keyboard_code_conversion.cc
diff --git a/ui/events/keycodes/keyboard_code_conversion.cc b/ui/events/keycodes/keyboard_code_conversion.cc
index b71dd6f0b94bd8600a55d596e574e0442e4f0c37..e31512b8c5e90efa0e642faead98db3f4f8cbf52 100644
--- a/ui/events/keycodes/keyboard_code_conversion.cc
+++ b/ui/events/keycodes/keyboard_code_conversion.cc
@@ -4,9 +4,12 @@
#include "ui/events/keycodes/keyboard_code_conversion.h"
+#include <algorithm>
+
#include "ui/events/event_constants.h"
#include "ui/events/keycodes/dom3/dom_code.h"
#include "ui/events/keycodes/dom3/dom_key.h"
+#include "ui/events/keycodes/dom_us_layout_data.h"
namespace ui {
@@ -130,6 +133,29 @@ bool IsRightSideDomCode(DomCode code) {
(code == DomCode::ALT_RIGHT) || (code == DomCode::OS_RIGHT);
}
+bool IsModifierDomCode(DomCode code) {
+ return (code == DomCode::CONTROL_LEFT) || (code == DomCode::CONTROL_RIGHT) ||
+ (code == DomCode::SHIFT_LEFT) || (code == DomCode::SHIFT_RIGHT) ||
+ (code == DomCode::ALT_LEFT) || (code == DomCode::ALT_RIGHT) ||
+ (code == DomCode::OS_LEFT) || (code == DomCode::OS_RIGHT);
+}
+
+// Returns the Windows-based VKEY value corresponding to a DOM Level 3 |code|,
+// assuming a base US English layout. The returned VKEY is located
+// (e.g. VKEY_LSHIFT).
+KeyboardCode DomCodeToUsLayoutKeyboardCode(DomCode dom_code) {
+ const DomCodeToKeyboardCodeEntry* end =
+ kDomCodeToKeyboardCodeMap + arraysize(kDomCodeToKeyboardCodeMap);
+ const DomCodeToKeyboardCodeEntry* found =
+ std::lower_bound(kDomCodeToKeyboardCodeMap, end, dom_code,
+ [](const DomCodeToKeyboardCodeEntry& a, DomCode b) {
+ return static_cast<int>(a.dom_code) < static_cast<int>(b);
+ });
+ if ((found != end) && (found->dom_code == dom_code))
+ return found->key_code;
+ return VKEY_UNKNOWN;
+}
+
} // anonymous namespace
base::char16 GetCharacterFromKeyCode(KeyboardCode key_code, int flags) {
@@ -267,6 +293,151 @@ bool GetMeaningFromKeyCode(KeyboardCode key_code,
return false;
}
+bool DomCodeToUsLayoutMeaning(DomCode dom_code,
+ int flags,
+ DomKey* out_dom_key,
+ base::char16* out_character,
+ KeyboardCode* out_key_code) {
+ if ((flags & EF_CONTROL_DOWN) == EF_CONTROL_DOWN) {
+ if (DomCodeToControlCharacter(dom_code, flags, out_dom_key, out_character,
+ out_key_code)) {
+ return true;
+ }
+ if (!IsModifierDomCode(dom_code)) {
+ *out_dom_key = DomKey::UNIDENTIFIED;
+ *out_character = 0;
+ *out_key_code = LocatedToNonLocatedKeyboardCode(
+ DomCodeToUsLayoutKeyboardCode(dom_code));
+ return true;
+ }
+ } else {
+ for (const auto& it : kPrintableCodeMap) {
+ if (it.dom_code == dom_code) {
+ int state = ((flags & EF_SHIFT_DOWN) == EF_SHIFT_DOWN);
+ base::char16 ch = it.character[state];
+ *out_dom_key = DomKey::CHARACTER;
+ *out_character = ch;
+ if ((flags & EF_CAPS_LOCK_DOWN) == EF_CAPS_LOCK_DOWN) {
+ ch |= 0x20;
+ if ((ch >= 'a') && (ch <= 'z'))
+ *out_character = it.character[state ^ 1];
+ }
+ *out_key_code = LocatedToNonLocatedKeyboardCode(
+ DomCodeToUsLayoutKeyboardCode(dom_code));
+ return true;
+ }
+ }
+ }
+ for (const auto& it : kNonPrintableCodeMap) {
+ if (it.dom_code == dom_code) {
+ *out_dom_key = it.dom_key;
+ *out_character = it.character;
+ *out_key_code = NonPrintableDomKeyToKeyboardCode(it.dom_key);
+ return true;
+ }
+ }
+ return false;
+}
+
+bool DomCodeToControlCharacter(DomCode dom_code,
+ int flags,
+ DomKey* dom_key,
+ base::char16* character,
+ KeyboardCode* key_code) {
+ if ((flags & EF_CONTROL_DOWN) == 0)
+ return false;
+
+ int code = static_cast<int>(dom_code);
+ const int kKeyA = static_cast<int>(DomCode::KEY_A);
+ // Control-A - Control-Z map to 0x01 - 0x1A.
+ if (code >= kKeyA && code <= static_cast<int>(DomCode::KEY_Z)) {
+ *character = static_cast<base::char16>(code - kKeyA + 1);
+ switch (dom_code) {
+ case DomCode::KEY_H:
+ *dom_key = DomKey::BACKSPACE;
+ *key_code = VKEY_BACK;
+ break;
+ case DomCode::KEY_I:
+ *dom_key = DomKey::TAB;
+ *key_code = VKEY_TAB;
+ break;
+ case DomCode::KEY_M:
+ *dom_key = DomKey::ENTER;
+ *key_code = VKEY_RETURN;
+ break;
+ default:
+ *dom_key = DomKey::CHARACTER;
+ *key_code = static_cast<KeyboardCode>(code - kKeyA + VKEY_A);
+ break;
+ }
+ return true;
+ }
+
+ if (flags & EF_SHIFT_DOWN) {
+ switch (dom_code) {
+ case DomCode::DIGIT2:
+ // NUL
+ *character = 0;
+ *dom_key = DomKey::CHARACTER;
+ *key_code = VKEY_2;
+ return true;
+ case DomCode::DIGIT6:
+ // RS
+ *character = 0x1E;
+ *dom_key = DomKey::CHARACTER;
+ *key_code = VKEY_6;
+ return true;
+ case DomCode::MINUS:
+ // US
+ *character = 0x1F;
+ *dom_key = DomKey::CHARACTER;
+ *key_code = VKEY_OEM_MINUS;
+ return true;
+ default:
+ return false;
+ }
+ }
+
+ switch (dom_code) {
+ case DomCode::ENTER:
+ // NL
+ *character = 0x0A;
+ *dom_key = DomKey::CHARACTER;
+ *key_code = VKEY_RETURN;
+ return true;
+ case DomCode::BRACKET_LEFT:
+ // ESC
+ *character = 0x1B;
+ *dom_key = DomKey::ESCAPE;
+ *key_code = VKEY_OEM_4;
+ return true;
+ case DomCode::BACKSLASH:
+ // FS
+ *character = 0x1C;
+ *dom_key = DomKey::CHARACTER;
+ *key_code = VKEY_OEM_5;
+ return true;
+ case DomCode::BRACKET_RIGHT:
+ // GS
+ *character = 0x1D;
+ *dom_key = DomKey::CHARACTER;
+ *key_code = VKEY_OEM_6;
+ return true;
+ default:
+ return false;
+ }
+}
+
+// Returns a Windows-based VKEY for a non-printable DOM Level 3 |key|.
+// The returned VKEY is non-positional (e.g. VKEY_SHIFT).
+KeyboardCode NonPrintableDomKeyToKeyboardCode(DomKey dom_key) {
+ for (const auto& it : kDomKeyToKeyboardCodeMap) {
+ if (it.dom_key == dom_key)
+ return it.key_code;
+ }
+ return VKEY_UNKNOWN;
+}
+
// Determine the non-located VKEY corresponding to a located VKEY.
KeyboardCode LocatedToNonLocatedKeyboardCode(KeyboardCode key_code) {
switch (key_code) {
@@ -343,4 +514,17 @@ KeyboardCode NonLocatedToLocatedKeyboardCode(KeyboardCode key_code,
}
}
+DomCode UsLayoutKeyboardCodeToDomCode(KeyboardCode key_code) {
+ key_code = NonLocatedToLocatedKeyboardCode(key_code, DomCode::NONE);
+ for (const auto& it : kDomCodeToKeyboardCodeMap) {
+ if (it.key_code == key_code)
+ return it.dom_code;
+ }
+ for (const auto& it : kFallbackKeyboardCodeToDomCodeMap) {
+ if (it.key_code == key_code)
+ return it.dom_code;
+ }
+ return DomCode::NONE;
+}
+
} // namespace ui

Powered by Google App Engine
This is Rietveld 408576698