Moving input device hotplug event processing outside of ui/display

ui/events/device_data_manager.h

Index: ui/events/device_data_manager.h
diff --git a/ui/events/device_data_manager.h b/ui/events/device_data_manager.h
index c9322045423e7307d9b5d81a00bbb58ca1ca2e54..2bff102acd0892db34be646b3993f41552251a02 100644
--- a/ui/events/device_data_manager.h
+++ b/ui/events/device_data_manager.h
@@ -7,13 +7,19 @@
 
 #include <stdint.h>
 
+#include <vector>
+
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/observer_list.h"
 #include "ui/events/events_base_export.h"
+#include "ui/events/touchscreen_device.h"
 #include "ui/gfx/transform.h"
 
 namespace ui {
 
+class InputDeviceEventObserver;
+
 // Keeps track of device mappings and event transformations.
 class EVENTS_BASE_EXPORT DeviceDataManager {
  public:
@@ -28,6 +34,11 @@ class EVENTS_BASE_EXPORT DeviceDataManager {
   void ApplyTouchTransformer(int touch_device_id, float* x, float* y);
   int64_t GetDisplayForTouchDevice(int touch_device_id) const;
 
+  void AddObserver(InputDeviceEventObserver* observer);
+  void RemoveObserver(InputDeviceEventObserver* observer);
+
+  virtual std::vector<TouchscreenDevice> GetTouchscreenDevices();

[Daniel Erat, 2014/06/17 21:59:18]

i think that there's still a general push to avoid non-interface inheritance in
chromium. how hard/messy would it be to restructure this class to e.g. be a pure
interface or call out to a passed-in object to get the touchscreen devices?

[dnicoara, 2014/06/20 17:01:05]

I've been talking to sadrul@ and rjkroege@ a bit about DDM. The plan is to mold
DDM into a nicer interface to allow Ozone to hook into it. Rob is going to add
better Ozone support, so I think I'll wait until he makes more progress before
getting this CL into its final form. (I'll work on updating the unittests in the
meantime)

+
  protected:
   DeviceDataManager();
 
@@ -42,6 +53,8 @@ class EVENTS_BASE_EXPORT DeviceDataManager {
   // Index table to find the TouchTransformer for a touch device.
   gfx::Transform touch_device_transformer_map_[kMaxDeviceNum];
 
+  ObserverList<InputDeviceEventObserver> observers_;
+
   DISALLOW_COPY_AND_ASSIGN(DeviceDataManager);
 };