ozone: evdev: Add gesture property provider

ui/events/ozone/evdev/event_factory_evdev.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/ozone/evdev/event_factory_evdev.h"
 
 #include <fcntl.h>
 #include <linux/input.h>
 
 #include "base/debug/trace_event.h"
 #include "base/stl_util.h"
 #include "base/task_runner.h"
 #include "base/thread_task_runner_handle.h"
 #include "base/threading/worker_pool.h"
 #include "ui/events/device_data_manager.h"
 #include "ui/events/ozone/device/device_event.h"
 #include "ui/events/ozone/device/device_manager.h"
 #include "ui/events/ozone/evdev/cursor_delegate_evdev.h"
 #include "ui/events/ozone/evdev/event_device_info.h"
 #include "ui/events/ozone/evdev/key_event_converter_evdev.h"
 #include "ui/events/ozone/evdev/touch_event_converter_evdev.h"
 
 #if defined(USE_EVDEV_GESTURES)
 #include "ui/events/ozone/evdev/libgestures_glue/event_reader_libevdev_cros.h"
 #include "ui/events/ozone/evdev/libgestures_glue/gesture_interpreter_libevdev_cros.h"
+#include "ui/events/ozone/evdev/libgestures_glue/gesture_property_provider.h"
 #endif
 
 #ifndef EVIOCSCLOCKID
 #define EVIOCSCLOCKID  _IOW('E', 0xa0, int)
 #endif
 
 namespace ui {
 
 namespace {
 
+// Max number of devices that we track.
+const int kMaxDeviceNum = 0xffff;
+
 typedef base::Callback<void(scoped_ptr<EventConverterEvdev>)>
     OpenInputDeviceReplyCallback;
 
 struct OpenInputDeviceParams {
   // Unique identifier for the new device.
   int id;
 
   // Device path to open.
   base::FilePath path;
 
   // Callback for dispatching events. Call on UI thread only.
   EventDispatchCallback dispatch_callback;
 
   // State shared between devices. Must not be dereferenced on worker thread.
   EventModifiersEvdev* modifiers;
   CursorDelegateEvdev* cursor;
+  GesturePropertyProvider* gesture_property_provider;
 };
 
 #if defined(USE_EVDEV_GESTURES)
 bool UseGesturesLibraryForDevice(const EventDeviceInfo& devinfo) {
   if (devinfo.HasAbsXY() && !devinfo.IsMappedToScreen())
     return true;  // touchpad
 
   if (devinfo.HasRelXY())
     return true;  // mouse
 
   return false;
 }
 #endif
 
 scoped_ptr<EventConverterEvdev> CreateConverter(
     const OpenInputDeviceParams& params,
     int fd,
     const EventDeviceInfo& devinfo) {
 #if defined(USE_EVDEV_GESTURES)
   // Touchpad or mouse: use gestures library.
   // EventReaderLibevdevCros -> GestureInterpreterLibevdevCros -> DispatchEvent
   if (UseGesturesLibraryForDevice(devinfo)) {
     scoped_ptr<GestureInterpreterLibevdevCros> gesture_interp =
         make_scoped_ptr(new GestureInterpreterLibevdevCros(
-            params.modifiers, params.cursor, params.dispatch_callback));
+            params.id, params.modifiers, params.cursor,
+            params.gesture_property_provider, params.dispatch_callback));
     scoped_ptr<EventReaderLibevdevCros> libevdev_reader =
         make_scoped_ptr(new EventReaderLibevdevCros(
             fd,
             params.path,
             params.id,
             gesture_interp.PassAs<EventReaderLibevdevCros::Delegate>()));
     return libevdev_reader.PassAs<EventConverterEvdev>();
   }
 #endif
 
@@ skipping 55 matching lines @@
 void CloseInputDevice(const base::FilePath& path,
                       scoped_ptr<EventConverterEvdev> converter) {
   TRACE_EVENT1("ozone", "CloseInputDevice", "path", path.value());
   converter.reset();
 }
 
 }  // namespace
 
 EventFactoryEvdev::EventFactoryEvdev(CursorDelegateEvdev* cursor,
                                      DeviceManager* device_manager)
-    : last_device_id_(0),
+    : device_id_counter_(0),
       device_manager_(device_manager),
       cursor_(cursor),
+#if defined(USE_EVDEV_GESTURES)
+      gesture_property_provider_(new GesturePropertyProvider),
+#endif
       dispatch_callback_(
           base::Bind(base::IgnoreResult(&EventFactoryEvdev::DispatchUiEvent),
                      base::Unretained(this))),
       weak_ptr_factory_(this) {
   DCHECK(device_manager_);
 }
 
 EventFactoryEvdev::~EventFactoryEvdev() { STLDeleteValues(&converters_); }
 
 void EventFactoryEvdev::DispatchUiEvent(Event* event) {
@@ skipping 22 matching lines @@
 void EventFactoryEvdev::OnDeviceEvent(const DeviceEvent& event) {
   if (event.device_type() != DeviceEvent::INPUT)
     return;
 
   switch (event.action_type()) {
     case DeviceEvent::ADD:
     case DeviceEvent::CHANGE: {
       TRACE_EVENT1("ozone", "OnDeviceAdded", "path", event.path().value());
 
       scoped_ptr<OpenInputDeviceParams> params(new OpenInputDeviceParams);
-      params->id = NextDeviceId();
+      params->id = GenerateDeviceId(event.path());
+      if (params->id < 0)
+        break;
       params->path = event.path();
       params->dispatch_callback = dispatch_callback_;
       params->modifiers = &modifiers_;
       params->cursor = cursor_;
+      params->gesture_property_provider = gesture_property_provider_.get();
 
       OpenInputDeviceReplyCallback reply_callback =
           base::Bind(&EventFactoryEvdev::AttachInputDevice,
                      weak_ptr_factory_.GetWeakPtr());
 
       // Dispatch task to open from the worker pool, since open may block.
       base::WorkerPool::PostTask(FROM_HERE,
                                  base::Bind(&OpenInputDevice,
                                             base::Passed(&params),
                                             ui_task_runner_,
                                             reply_callback),
                                  true /* task_is_slow */);
     }
       break;
     case DeviceEvent::REMOVE: {
       TRACE_EVENT1("ozone", "OnDeviceRemoved", "path", event.path().value());
+      RevokeDeviceId(event.path());
       DetachInputDevice(event.path());
     }
       break;
   }
 }
 
 void EventFactoryEvdev::OnDispatcherListChanged() {
   if (!ui_task_runner_.get()) {
     ui_task_runner_ = base::ThreadTaskRunnerHandle::Get();
     // Scan & monitor devices.
@@ skipping 51 matching lines @@
     if (it->second->HasTouchscreen()) {
       touchscreens.push_back(TouchscreenDevice(it->second->id(),
                                                it->second->GetTouchscreenSize(),
                                                false /* is_internal */));
     }
   }
 
   observer->OnTouchscreenDevicesUpdated(touchscreens);
 }
 
-int EventFactoryEvdev::NextDeviceId() {
-  return ++last_device_id_;
+int EventFactoryEvdev::GenerateDeviceId(const base::FilePath& file_path) {
+  // Do not generate if the maximum allowed device number is reached.
+  if (used_device_ids_.size() >= kMaxDeviceNum)
+    return -1;
+
+  // Generate a new device id for use.
+  int id = device_id_counter_;
+  while (used_device_ids_.find(id) != used_device_ids_.end())
+    id = (id + 1) & kMaxDeviceNum;

[spang, 2014/10/15 11:36:41]

This is way overkill. If 2 billion devices over the lifetime of the browser
isn't enough, we can solve that by switching to int64_t. Realistically, I think
int is plenty - let's keep it simple.

So: All these changes to ID generation are unnecessary for this CL, please drop
them.

[Shecky Lin, 2014/10/16 04:53:53]

Done.

+  device_id_counter_ = (id + 1) & kMaxDeviceNum;
+  used_device_ids_.insert(id);
+
+  // If a same device node is opened again, we just replace its old id with a
+  // new one as this is the behavior we use for converters as well.
+  device_ids_map_[file_path] = id;
+  return id;
+}
+
+void EventFactoryEvdev::RevokeDeviceId(const base::FilePath& file_path) {
+  DeviceIdMap::iterator it = device_ids_map_.find(file_path);
+  if (it == device_ids_map_.end())
+    return;
+  used_device_ids_.erase(it->second);
+  device_ids_map_.erase(it);
 }
 
 }  // namespace ui