Fixed the Touchscreen.TouchEventsEnabled histogram to record the correct values on X11 and Ozone ba…

ui/events/devices/device_data_manager.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/events/devices/device_data_manager.h"
 
 #include "base/at_exit.h"
 #include "base/bind.h"
 #include "base/logging.h"
 #include "ui/events/devices/input_device_event_observer.h"
 #include "ui/gfx/display.h"
 #include "ui/gfx/geometry/point3_f.h"
 
 namespace ui {
 
 namespace {
 
 bool InputDeviceEquals(const ui::InputDevice& a, const ui::InputDevice& b) {
   return a.id == b.id;
 }
 
 }  // namespace
 
 // static
 DeviceDataManager* DeviceDataManager::instance_ = NULL;
 
 DeviceDataManager::DeviceDataManager() {
-  CHECK(!instance_) << "Can not create multiple instances of DeviceDataManager";
-  instance_ = this;
-
-  base::AtExitManager::RegisterTask(
-      base::Bind(&base::DeletePointer<DeviceDataManager>, this));
-
   ClearTouchDeviceAssociations();
 }
 
 DeviceDataManager::~DeviceDataManager() {
-  CHECK_EQ(this, instance_);
-  instance_ = NULL;
 }
 
 // static
 DeviceDataManager* DeviceDataManager::instance() { return instance_; }
 
+void DeviceDataManager::set_instance(DeviceDataManager* instance) {
+  instance_ = instance;
+}
+
 // static
 void DeviceDataManager::CreateInstance() {
   if (instance())
     return;
 
-  new DeviceDataManager();
+  CHECK(!instance_) << "Can not create multiple instances of DeviceDataManager";
+  instance_ = new DeviceDataManager();
+
+  base::AtExitManager::RegisterTask(base::Bind(DeleteInstance));
+}
+
+void DeviceDataManager::DeleteInstance() {
+  if (instance_) {
+    delete instance_;
+    instance_ = NULL;
+  }
 }
 
 // static
 DeviceDataManager* DeviceDataManager::GetInstance() {
   CHECK(instance_) << "DeviceDataManager was not created.";
   return instance_;
 }
 
 // static
 bool DeviceDataManager::HasInstance() {
@@ skipping 58 matching lines @@
 void DeviceDataManager::OnTouchscreenDevicesUpdated(
     const std::vector<TouchscreenDevice>& devices) {
   if (devices.size() == touchscreen_devices_.size() &&
       std::equal(devices.begin(),
                  devices.end(),
                  touchscreen_devices_.begin(),
                  InputDeviceEquals)) {
     return;
   }
   touchscreen_devices_ = devices;
-  FOR_EACH_OBSERVER(InputDeviceEventObserver,
-                    observers_,
-                    OnTouchscreenDeviceConfigurationChanged());
+  NotifyObserversTouchscreenDeviceConfigurationChanged();
 }
 
 void DeviceDataManager::OnKeyboardDevicesUpdated(
     const std::vector<KeyboardDevice>& devices) {
   if (devices.size() == keyboard_devices_.size() &&
       std::equal(devices.begin(),
                  devices.end(),
                  keyboard_devices_.begin(),
                  InputDeviceEquals)) {
     return;
   }
   keyboard_devices_ = devices;
-  FOR_EACH_OBSERVER(InputDeviceEventObserver,
-                    observers_,
-                    OnKeyboardDeviceConfigurationChanged());
+  NotifyObserversKeyboardDeviceConfigurationChanged();
 }
 
 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());
+  NotifyObserversMouseDeviceConfigurationChanged();
 }
 
 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());
+  NotifyObserversTouchpadDeviceConfigurationChanged();
 }
 
 void DeviceDataManager::OnDeviceListsComplete() {
   if (!device_lists_complete_) {
     device_lists_complete_ = true;
-    FOR_EACH_OBSERVER(InputDeviceEventObserver, observers_,
-                      OnDeviceListsComplete());
+    NotifyObserversDeviceListsComplete();
   }
 }
 
+void DeviceDataManager::NotifyObserversTouchscreenDeviceConfigurationChanged() {
+  FOR_EACH_OBSERVER(InputDeviceEventObserver, observers_,
+                    OnTouchscreenDeviceConfigurationChanged());
+}
+
+void DeviceDataManager::NotifyObserversKeyboardDeviceConfigurationChanged() {
+  FOR_EACH_OBSERVER(InputDeviceEventObserver, observers_,
+                    OnKeyboardDeviceConfigurationChanged());
+}
+
+void DeviceDataManager::NotifyObserversMouseDeviceConfigurationChanged() {
+  FOR_EACH_OBSERVER(InputDeviceEventObserver, observers_,
+                    OnMouseDeviceConfigurationChanged());
+}
+
+void DeviceDataManager::NotifyObserversTouchpadDeviceConfigurationChanged() {
+  FOR_EACH_OBSERVER(InputDeviceEventObserver, observers_,
+                    OnTouchpadDeviceConfigurationChanged());
+}
+
+void DeviceDataManager::NotifyObserversDeviceListsComplete() {
+    FOR_EACH_OBSERVER(InputDeviceEventObserver, observers_,
+                      OnDeviceListsComplete());
+}
+
 void DeviceDataManager::AddObserver(InputDeviceEventObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void DeviceDataManager::RemoveObserver(InputDeviceEventObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
 }  // namespace ui