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

Side by Side Diff: ui/events/keycodes/keyboard_code_conversion.h

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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef UI_EVENTS_KEYCODES_KEYBOARD_CODE_CONVERSION_H_ 5 #ifndef UI_EVENTS_KEYCODES_KEYBOARD_CODE_CONVERSION_H_
6 #define UI_EVENTS_KEYCODES_KEYBOARD_CODE_CONVERSION_H_ 6 #define UI_EVENTS_KEYCODES_KEYBOARD_CODE_CONVERSION_H_
7 7
8 #include "base/compiler_specific.h"
8 #include "base/strings/string16.h" 9 #include "base/strings/string16.h"
9 #include "ui/events/events_base_export.h" 10 #include "ui/events/events_base_export.h"
10 #include "ui/events/keycodes/keyboard_codes.h" 11 #include "ui/events/keycodes/keyboard_codes.h"
11 12
12 namespace ui { 13 namespace ui {
13 14
14 enum class DomCode; 15 enum class DomCode;
15 enum class DomKey; 16 enum class DomKey;
16 17
17 // Helper functions to get the meaning of a Windows key code in a 18 // Helper functions to get the meaning of a Windows key code in a
18 // platform independent way. It supports control characters as well. 19 // platform independent way. It supports control characters as well.
19 // It assumes a US keyboard layout is used, so it may only be used when there 20 // It assumes a US keyboard layout is used, so it may only be used when there
20 // is no native event or no better way to get the character. 21 // is no native event or no better way to get the character.
22 //
21 // For example, if a virtual keyboard implementation can only generate key 23 // For example, if a virtual keyboard implementation can only generate key
22 // events with key_code and flags information, then there is no way for us to 24 // events with key_code and flags information, then there is no way for us to
23 // determine the actual character that should be generate by the key. Because 25 // determine the actual character that should be generate by the key. Because
24 // a key_code only represents a physical key on the keyboard, it has nothing 26 // a key_code only represents a physical key on the keyboard, it has nothing
25 // to do with the actual character printed on that key. In such case, the only 27 // to do with the actual character printed on that key. In such case, the only
26 // thing we can do is to assume that we are using a US keyboard and get the 28 // thing we can do is to assume that we are using a US keyboard and get the
27 // character according to US keyboard layout definition. 29 // character according to US keyboard layout definition. Preferably, such
28 // If a virtual keyboard implementation wants to support other keyboard 30 // events should be created using a full KeyEvent constructor, explicitly
29 // layouts, that may generate different text for a certain key than on a US 31 // specifying the character and DOM 3 values as well as the legacy VKEY.
30 // keyboard, a special native event object should be introduced to carry extra 32 //
31 // information to help determine the correct character. 33 // TODO(kpschoedel): replace remaining uses of the ...FromKeyCode() functions
32 // Take XKeyEvent as an example, it contains not only keycode and modifier 34 // and remove them, to avoid future creation of underspecified key events.
33 // flags but also group and other extra XKB information to help determine the 35 // crbug.com/444045
34 // correct character. That's why we can use XLookupString() function to get
35 // the correct text generated by a X key event (See how is GetCharacter()
36 // implemented in event_x.cc).
37 EVENTS_BASE_EXPORT base::char16 GetCharacterFromKeyCode(KeyboardCode key_code, 36 EVENTS_BASE_EXPORT base::char16 GetCharacterFromKeyCode(KeyboardCode key_code,
38 int flags); 37 int flags);
39 EVENTS_BASE_EXPORT bool GetMeaningFromKeyCode(KeyboardCode key_code, 38 EVENTS_BASE_EXPORT bool GetMeaningFromKeyCode(KeyboardCode key_code,
40 int flags, 39 int flags,
41 DomKey* dom_key, 40 DomKey* dom_key,
42 base::char16* character); 41 base::char16* character);
43 42
43 // Helper function to map a physical key state (dom_code and flags)
44 // to a meaning (dom_key and character, together corresponding to the
45 // DOM keyboard event |key| value), along with a corresponding Windows-based
46 // key_code.
47 //
48 // This follows a US keyboard layout, so it should only be used when there
49 // is no other better way to obtain the meaning (e.g. actual keyboard layout).
50 // Returns true and sets the output parameters if the (dom_code, flags) pair
51 // has an interpretation in the US English layout; otherwise the output
52 // parameters are untouched.
53 EVENTS_BASE_EXPORT bool DomCodeToUsLayoutMeaning(DomCode dom_code,
54 int flags,
55 DomKey* dom_key,
56 base::char16* character,
57 KeyboardCode* key_code)
58 WARN_UNUSED_RESULT;
59
60 // Obtains the control character corresponding to a physical key;
61 // that is, the meaning of the physical key state (dom_code, and flags
62 // containing EF_CONTROL_DOWN) under the base US English layout.
63 // Returns true and sets the output parameters if the (dom_code, flags) pair
64 // is interpreted as a control character; otherwise the output parameters
65 // are untouched.
66 EVENTS_BASE_EXPORT bool DomCodeToControlCharacter(DomCode dom_code,
67 int flags,
68 DomKey* dom_key,
69 base::char16* character,
70 KeyboardCode* key_code)
71 WARN_UNUSED_RESULT;
72
73 // Returns a Windows-based VKEY for a non-printable DOM Level 3 |key|.
74 // The returned VKEY is non-located (e.g. VKEY_SHIFT).
75 EVENTS_BASE_EXPORT KeyboardCode
76 NonPrintableDomKeyToKeyboardCode(DomKey dom_key);
77
44 // Determine the non-located VKEY corresponding to a located VKEY. 78 // Determine the non-located VKEY corresponding to a located VKEY.
45 // Most modifier keys have two kinds of KeyboardCode: located (e.g. 79 // Most modifier keys have two kinds of KeyboardCode: located (e.g.
46 // VKEY_LSHIFT and VKEY_RSHIFT), that indentify one of two specific 80 // VKEY_LSHIFT and VKEY_RSHIFT), that indentify one of two specific
47 // physical keys, and non-located (e.g. VKEY_SHIFT) that identify 81 // physical keys, and non-located (e.g. VKEY_SHIFT) that identify
48 // only the operation. 82 // only the operation.
49 EVENTS_BASE_EXPORT KeyboardCode 83 EVENTS_BASE_EXPORT KeyboardCode
50 LocatedToNonLocatedKeyboardCode(KeyboardCode key_code); 84 LocatedToNonLocatedKeyboardCode(KeyboardCode key_code);
51 85
52 // Determine the located VKEY corresponding to a non-located VKEY. 86 // Determine the located VKEY corresponding to a non-located VKEY.
53 EVENTS_BASE_EXPORT KeyboardCode 87 EVENTS_BASE_EXPORT KeyboardCode
54 NonLocatedToLocatedKeyboardCode(KeyboardCode key_code, DomCode dom_code); 88 NonLocatedToLocatedKeyboardCode(KeyboardCode key_code, DomCode dom_code);
55 89
90 // Returns a DOM Level 3 |code| from a Windows-based VKEY value.
91 // This assumes a US layout and should only be used when |code| cannot be
92 // determined from a physical scan code, for example when a key event was
93 // generated synthetically by JavaScript with only a VKEY value supplied.
94 EVENTS_BASE_EXPORT DomCode UsLayoutKeyboardCodeToDomCode(KeyboardCode key_code);
95
56 } // namespace ui 96 } // namespace ui
57 97
58 #endif // UI_EVENTS_KEYCODES_KEYBOARD_CODE_CONVERSION_H_ 98 #endif // UI_EVENTS_KEYCODES_KEYBOARD_CODE_CONVERSION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698