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

Unified Diff: ui/events/platform/x11/x11_hotplug_event_handler.cc

Issue 889403003: Make PointerDeviceObserver work under Ozone (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/platform/x11/x11_hotplug_event_handler.cc
diff --git a/ui/events/platform/x11/x11_hotplug_event_handler.cc b/ui/events/platform/x11/x11_hotplug_event_handler.cc
index 78ecf3554608f37a5385f6491f3bad718b3912a7..4e546f9ebd43405892a0d3b871422dfc081f9bbd 100644
--- a/ui/events/platform/x11/x11_hotplug_event_handler.cc
+++ b/ui/events/platform/x11/x11_hotplug_event_handler.cc
@@ -340,6 +340,8 @@ void X11HotplugEventHandler::OnHotplugEvent() {
UiCallbacks callbacks;
callbacks.keyboard_callback = base::Bind(&OnKeyboardDevices);
callbacks.touchscreen_callback = base::Bind(&OnTouchscreenDevices);
+ // TODO(pkotwicz): Requery the lists of mice and touchpads and send the new
+ // lists to DeviceHotplugEventObserver.
sadrul 2015/02/03 15:29:04 Would we need to requery the list here? Presumably
sadrul 2015/02/03 17:33:04 ping
pkotwicz 2015/02/03 18:09:33 I have updated the comment so that it is clearer.
// Parsing the device information may block, so delegate the operation to a
// worker thread. Once the device information is extracted the parsed devices
« ui/events/devices/device_data_manager.h ('K') | « ui/events/ozone/evdev/event_factory_evdev.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698