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

Unified Diff: ui/events/devices/device_data_manager.cc

Issue 946173002: Make PointerDeviceObserver use ui::InputDeviceEventObserver on X11 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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/devices/device_data_manager.cc
diff --git a/ui/events/devices/device_data_manager.cc b/ui/events/devices/device_data_manager.cc
index eb96c46478924f00385b6a865bb33b1580ac7411..fdf99563f55db76ad6dab6cace32a3058c4f5d6d 100644
--- a/ui/events/devices/device_data_manager.cc
+++ b/ui/events/devices/device_data_manager.cc
@@ -151,6 +151,14 @@ void DeviceDataManager::OnKeyboardDevicesUpdated(
void DeviceDataManager::OnMouseDevicesUpdated(
const std::vector<InputDevice>& devices) {
+ if (devices.size() == mouse_devices_.size() &&
+ std::equal(devices.begin(),
+ devices.end(),
+ mouse_devices_.begin(),
+ InputDeviceEquals)) {
+ return;
+ }
+ mouse_devices_ = devices;
FOR_EACH_OBSERVER(InputDeviceEventObserver,
observers_,
OnMouseDeviceConfigurationChanged());
@@ -158,6 +166,14 @@ void DeviceDataManager::OnMouseDevicesUpdated(
void DeviceDataManager::OnTouchpadDevicesUpdated(
const std::vector<InputDevice>& devices) {
+ if (devices.size() == touchpad_devices_.size() &&
+ std::equal(devices.begin(),
+ devices.end(),
+ touchpad_devices_.begin(),
+ InputDeviceEquals)) {
+ return;
+ }
+ touchpad_devices_ = devices;
FOR_EACH_OBSERVER(InputDeviceEventObserver,
observers_,
OnTouchpadDeviceConfigurationChanged());

Powered by Google App Engine
This is Rietveld 408576698