HID backend

device/hid/hid_service_linux.h

+// Copyright (c) 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 DEVICE_HID_HID_SERVICE_LINUX_H_
+#define DEVICE_HID_HID_SERVICE_LINUX_H_
+
+#include <libudev.h>
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/message_loop/message_pump_libevent.h"
+#include "device/hid/hid_device_info.h"
+#include "device/hid/hid_service.h"
+
+struct udev_device;
+
+namespace device {
+
+class HidConnection;
+
+template<typename T, void func(T*)>

[Ken Rockot(use gerrit already), 2014/01/21 23:03:39]

Can this struct and all the typedefs instead be private to HidServiceLinux?

[Bei Zhang, 2014/01/24 12:48:47]

I believe the whole file is pretty private. Since they are all typedefs and
template, no symbols are exported.

On 2014/01/21 23:03:39, Ken Rockot wrote:

+struct GeneralDeleter {

[Ken Rockot(use gerrit already), 2014/01/21 23:03:39]

naming nit: How about just Deleter? It's already general :)

[Bei Zhang, 2014/01/24 12:48:47]

Wonderful!!! Now I can fit everything in 80chs.

+  void operator()(T* enumerate) const {
+    if (enumerate != NULL)
+      func(enumerate);
+  }
+};
+
+typedef GeneralDeleter<udev_enumerate, udev_enumerate_unref>
+    UdevEnumerateDeleter;
+typedef GeneralDeleter<udev_device, udev_device_unref> UdevDeviceDeleter;
+typedef GeneralDeleter<udev, udev_unref> UdevDeleter;
+typedef GeneralDeleter<udev_monitor, udev_monitor_unref> UdevMonitorDeleter;
+
+typedef scoped_ptr<udev_device, UdevDeviceDeleter> ScopedUdevDevicePtr;
+typedef scoped_ptr<udev_enumerate, UdevEnumerateDeleter>
+    ScopedUdevEnumeratePtr;
+
+class HidServiceLinux : public HidService,
+                        public base::MessagePumpLibevent::Watcher {
+ public:
+  HidServiceLinux();
+
+  virtual scoped_refptr<HidConnection> Connect(std::string device_id) OVERRIDE;
+
+  // Implements base::MessagePumpLibevent::Watcher
+  virtual void OnFileCanReadWithoutBlocking(int fd) OVERRIDE;
+  virtual void OnFileCanWriteWithoutBlocking(int fd) OVERRIDE;
+
+ private:
+  virtual ~HidServiceLinux();
+
+  void Enumerate();
+  void PlatformDeviceAdd(udev_device* device);
+  void PlatformDeviceRemove(udev_device* raw_dev);
+
+  scoped_ptr<udev, UdevDeleter> udev_;
+  scoped_ptr<udev_monitor, UdevMonitorDeleter> monitor_;
+  int monitor_fd_;
+  base::MessagePumpLibevent::FileDescriptorWatcher monitor_watcher_;
+
+  DISALLOW_COPY_AND_ASSIGN(HidServiceLinux);
+};
+
+}  // namespace device
+
+#endif  // DEVICE_HID_HID_SERVICE_LINUX_H_