Extract touchscreen device management into a generic manager

ui/events/device_data_manager.h

+// 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.
+
+#ifndef UI_EVENTS_DEVICE_DATA_MANAGER_H_
+#define UI_EVENTS_DEVICE_DATA_MANAGER_H_
+
+#include <stdint.h>
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "ui/events/events_base_export.h"
+#include "ui/gfx/transform.h"
+
+namespace ui {
+
+// Keeps track of device mappings and event transformations.
+class EVENTS_BASE_EXPORT DeviceDataManager {
+ public:
+  virtual ~DeviceDataManager();
+
+  static void CreateInstance();
+  static DeviceDataManager* GetInstance();
+
+  void ClearTouchTransformerRecord();
+  void UpdateTouchInfoForDisplay(int64_t display_id,
+                                 int touch_device_id,
+                                 const gfx::Transform& touch_transformer);
+  void ApplyTouchTransformer(int touch_device_id, float* x, float* y);
+  int64_t GetDisplayForTouchDevice(int touch_device_id) const;
+
+ protected:
+  DeviceDataManager();
+
+  static DeviceDataManager* instance_;
+
+  bool IsTouchDeviceIdValid(int touch_device_id) const;
+
+  static const int kMaxDeviceNum = 128;
+
+  // Table to keep track of which display id is mapped to which touch device.
+  int64_t touch_device_to_display_map_[kMaxDeviceNum];

[sadrul, 2014/06/20 13:37:52]

These should be privates. Have a protected static ::set_instance() (or just set
instance_ from the ctor).

I think kMaxDeviceNum can remain protected.

[dnicoara, 2014/06/20 14:48:00]

Done.

+  // Index table to find the TouchTransformer for a touch device.
+  gfx::Transform touch_device_transformer_map_[kMaxDeviceNum];
+
+  DISALLOW_COPY_AND_ASSIGN(DeviceDataManager);
+};
+
+}  // namespace ui
+
+#endif  // UI_EVENTS_DEVICE_DATA_MANAGER_H_