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

Unified Diff: ui/keyboard/keyboard_util.cc

Issue 1284433002: Revise ui::DomKey to unify character and non-character codes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: IsDead Created 5 years, 4 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
« no previous file with comments | « ui/events/ozone/layout/xkb/xkb_keyboard_layout_engine_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/keyboard/keyboard_util.cc
diff --git a/ui/keyboard/keyboard_util.cc b/ui/keyboard/keyboard_util.cc
index 188453ff7dc09ccc6c18f2a646640be989e871d0..572aecb92798ad6df80cfabd2c7751ea89042a92 100644
--- a/ui/keyboard/keyboard_util.cc
+++ b/ui/keyboard/keyboard_util.cc
@@ -38,7 +38,7 @@ const char kKeyUp[] = "keyup";
void SendProcessKeyEvent(ui::EventType type,
aura::WindowTreeHost* host) {
ui::KeyEvent event(type, ui::VKEY_PROCESSKEY, ui::DomCode::NONE,
- ui::EF_IS_SYNTHESIZED, ui::DomKey::PROCESS, 0,
+ ui::EF_IS_SYNTHESIZED, ui::DomKey::PROCESS,
ui::EventTimeForNow());
ui::EventDispatchDetails details =
host->event_processor()->OnEventFromSource(&event);
@@ -253,17 +253,16 @@ bool MoveCursor(int swipe_direction,
if (domcodex != ui::DomCode::NONE) {
ui::KeyboardCode codex = ui::VKEY_UNKNOWN;
ui::DomKey domkeyx = ui::DomKey::NONE;
- base::char16 cx;
- ignore_result(DomCodeToUsLayoutMeaning(domcodex, ui::EF_NONE, &domkeyx,
- &cx, &codex));
+ ignore_result(DomCodeToUsLayoutDomKey(domcodex, ui::EF_NONE, &domkeyx,
+ &codex));
ui::KeyEvent press_event(ui::ET_KEY_PRESSED, codex, domcodex,
- modifier_flags, domkeyx, cx,
+ modifier_flags, domkeyx,
ui::EventTimeForNow());
ui::EventDispatchDetails details =
host->event_processor()->OnEventFromSource(&press_event);
CHECK(!details.dispatcher_destroyed);
ui::KeyEvent release_event(ui::ET_KEY_RELEASED, codex, domcodex,
- modifier_flags, domkeyx, cx,
+ modifier_flags, domkeyx,
ui::EventTimeForNow());
details = host->event_processor()->OnEventFromSource(&release_event);
CHECK(!details.dispatcher_destroyed);
@@ -273,17 +272,16 @@ bool MoveCursor(int swipe_direction,
if (domcodey != ui::DomCode::NONE) {
ui::KeyboardCode codey = ui::VKEY_UNKNOWN;
ui::DomKey domkeyy = ui::DomKey::NONE;
- base::char16 cy;
- ignore_result(DomCodeToUsLayoutMeaning(domcodey, ui::EF_NONE, &domkeyy,
- &cy, &codey));
+ ignore_result(DomCodeToUsLayoutDomKey(domcodey, ui::EF_NONE, &domkeyy,
+ &codey));
ui::KeyEvent press_event(ui::ET_KEY_PRESSED, codey, domcodey,
- modifier_flags, domkeyy, cy,
+ modifier_flags, domkeyy,
ui::EventTimeForNow());
ui::EventDispatchDetails details =
host->event_processor()->OnEventFromSource(&press_event);
CHECK(!details.dispatcher_destroyed);
ui::KeyEvent release_event(ui::ET_KEY_RELEASED, codey, domcodey,
- modifier_flags, domkeyy, cy,
+ modifier_flags, domkeyy,
ui::EventTimeForNow());
details = host->event_processor()->OnEventFromSource(&release_event);
CHECK(!details.dispatcher_destroyed);
« no previous file with comments | « ui/events/ozone/layout/xkb/xkb_keyboard_layout_engine_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698