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

Unified Diff: ui/events/ozone/evdev/libgestures_glue/gesture_interpreter_libevdev_cros.cc

Issue 872883003: [PATCH 2/11] ozone: evdev: Move KeyboardEvdev usage during dispatch to EventFactoryEvdev (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: updates for events_unittests Created 5 years, 11 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/ozone/evdev/libgestures_glue/gesture_interpreter_libevdev_cros.cc
diff --git a/ui/events/ozone/evdev/libgestures_glue/gesture_interpreter_libevdev_cros.cc b/ui/events/ozone/evdev/libgestures_glue/gesture_interpreter_libevdev_cros.cc
index c338c5ba0450d3a97306bc8047294b3e32f524ac..3afd5ce1c9aa6d0dc1f62d1808fa073e06a68e8c 100644
--- a/ui/events/ozone/evdev/libgestures_glue/gesture_interpreter_libevdev_cros.cc
+++ b/ui/events/ozone/evdev/libgestures_glue/gesture_interpreter_libevdev_cros.cc
@@ -16,7 +16,7 @@
#include "ui/events/ozone/evdev/event_device_info.h"
#include "ui/events/ozone/evdev/event_device_util.h"
#include "ui/events/ozone/evdev/event_modifiers_evdev.h"
-#include "ui/events/ozone/evdev/keyboard_evdev.h"
+#include "ui/events/ozone/evdev/keyboard_util_evdev.h"
#include "ui/events/ozone/evdev/libgestures_glue/gesture_property_provider.h"
#include "ui/events/ozone/evdev/libgestures_glue/gesture_timer_provider.h"
#include "ui/events/ozone/evdev/mouse_button_map_evdev.h"
@@ -110,16 +110,16 @@ GestureInterpreterLibevdevCros::GestureInterpreterLibevdevCros(
EventModifiersEvdev* modifiers,
MouseButtonMapEvdev* button_map,
CursorDelegateEvdev* cursor,
- KeyboardEvdev* keyboard,
GesturePropertyProvider* property_provider,
+ const KeyEventDispatchCallback& key_callback,
const EventDispatchCallback& callback)
: id_(id),
is_mouse_(false),
modifiers_(modifiers),
button_map_(button_map),
cursor_(cursor),
- keyboard_(keyboard),
property_provider_(property_provider),
+ key_callback_(key_callback),
dispatch_callback_(callback),
interpreter_(NULL),
evdev_(NULL),
@@ -245,8 +245,8 @@ void GestureInterpreterLibevdevCros::SetAllowedKeys(
allowed_keys_.reset(new std::set<int>());
for (const auto& it : *allowed_keys) {
- int evdev_code = KeyboardEvdev::NativeCodeToEvdevCode(
- KeycodeConverter::DomCodeToNativeKeycode(it));
+ int evdev_code =
+ NativeCodeToEvdevCode(KeycodeConverter::DomCodeToNativeKeycode(it));
allowed_keys_->insert(evdev_code);
}
}
@@ -508,7 +508,7 @@ void GestureInterpreterLibevdevCros::DispatchChangedKeys(Evdev* evdev,
continue;
// Dispatch key press or release to keyboard.
- keyboard_->OnKeyChange(key, value);
+ key_callback_.Run(id_, key, value);
}
}

Powered by Google App Engine
This is Rietveld 408576698