Changed DeviceDataManager to be able to create multiple instances for the purposes of test.

The DeviceDataManager was implemented as a singleton and it was impossible
to create two different instances in tests. This change makes it possible
for future test framework to create multiple instances.

BUG=499476

Review URL: https://codereview.chromium.org/1202383002

Cr-Commit-Position: refs/heads/master@{#336558}

1 files changed, 15 insertions, 9 deletions

diff --git a/ui/events/devices/device_data_manager.cc b/ui/events/devices/device_data_manager.cc
index 44ca1fa..e725736 100644
--- a/ui/events/devices/device_data_manager.cc
+++ b/ui/events/devices/device_data_manager.cc
@@ -25,29 +25,35 @@ bool InputDeviceEquals(const ui::InputDevice& a, const ui::InputDevice& b) {
 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) {
+  CHECK(!instance_) << "Can not set multiple instances of DeviceDataManager";
+  instance_ = instance;
+}
+
 // static
 void DeviceDataManager::CreateInstance() {
   if (instance())
     return;
 
-  new DeviceDataManager();
+  set_instance(new DeviceDataManager());
+
+  base::AtExitManager::RegisterTask(base::Bind(DeleteInstance));
+}
+
+void DeviceDataManager::DeleteInstance() {
+  if (instance_) {
+    delete instance_;
+    instance_ = NULL;
+  }
 }
 
 // static