summaryrefslogtreecommitdiffstats
path: root/device/hid/hid_service_linux.cc
diff options
context:
space:
mode:
authorrockot@chromium.org <rockot@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-02-27 17:29:10 +0000
committerrockot@chromium.org <rockot@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-02-27 17:29:10 +0000
commit26958d1d63547e6ff5f9f254e718905709fb6daf (patch)
tree2cc5f9553d8bd8d56f8d438fec805e11c2e2e01c /device/hid/hid_service_linux.cc
parent76d03f895522ae0ea5106c31618115b1a1d2c575 (diff)
downloadchromium_src-26958d1d63547e6ff5f9f254e718905709fb6daf.zip
chromium_src-26958d1d63547e6ff5f9f254e718905709fb6daf.tar.gz
chromium_src-26958d1d63547e6ff5f9f254e718905709fb6daf.tar.bz2
Clean up HID backend and API.
The Mac backend no longer creates its own thread and instead simply enforces single-threaded usage on any thread which supports I/O. Ref management has also been sanitized. The Linux backend now implements feature report support. The backend interface no longer expects implicit report IDs in buffers passed to Write or SetFeatureReport. Instead these interfaces (as well as GetFeatureReport) take explicit report ID arguments. The API interface has been updated to reflect the improved report ID treatment. Finally, the API also now exposes opaque device identifiers on enumeration, rather than exposing raw system paths or other information that could be potentially sensitive. BUG=347294 Review URL: https://codereview.chromium.org/161823002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253853 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'device/hid/hid_service_linux.cc')
-rw-r--r--device/hid/hid_service_linux.cc98
1 files changed, 46 insertions, 52 deletions
diff --git a/device/hid/hid_service_linux.cc b/device/hid/hid_service_linux.cc
index f02a550..0c8d7c9 100644
--- a/device/hid/hid_service_linux.cc
+++ b/device/hid/hid_service_linux.cc
@@ -1,22 +1,19 @@
-// Copyright (c) 2014 The Chromium Authors. All rights reserved.
+// 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 <libudev.h>
+#include <stdint.h>
+
#include <string>
#include <vector>
-#include "base/basictypes.h"
-#include "base/bind.h"
-#include "base/callback.h"
#include "base/logging.h"
-#include "base/memory/scoped_vector.h"
#include "base/platform_file.h"
+#include "base/stl_util.h"
#include "base/strings/string_number_conversions.h"
#include "base/strings/string_piece.h"
#include "base/strings/string_split.h"
-#include "base/threading/thread_restrictions.h"
-#include "device/hid/hid_connection.h"
#include "device/hid/hid_connection_linux.h"
#include "device/hid/hid_device_info.h"
#include "device/hid/hid_service_linux.h"
@@ -79,6 +76,37 @@ HidServiceLinux::HidServiceLinux() {
Enumerate();
}
+scoped_refptr<HidConnection> HidServiceLinux::Connect(
+ const HidDeviceId& device_id) {
+ HidDeviceInfo device_info;
+ if (!GetDeviceInfo(device_id, &device_info))
+ return NULL;
+
+ ScopedUdevDevicePtr hid_device(
+ udev_device_new_from_syspath(udev_.get(), device_info.device_id.c_str()));
+ if (hid_device) {
+ return new HidConnectionLinux(device_info, hid_device.Pass());
+ }
+ return NULL;
+}
+
+void HidServiceLinux::OnFileCanReadWithoutBlocking(int fd) {
+ DCHECK_EQ(monitor_fd_, fd);
+
+ ScopedUdevDevicePtr dev(udev_monitor_receive_device(monitor_.get()));
+ if (!dev)
+ return;
+
+ std::string action(udev_device_get_action(dev.get()));
+ if (action == kUdevActionAdd) {
+ PlatformAddDevice(dev.get());
+ } else if (action == kUdevActionRemove) {
+ PlatformRemoveDevice(dev.get());
+ }
+}
+
+void HidServiceLinux::OnFileCanWriteWithoutBlocking(int fd) {}
+
HidServiceLinux::~HidServiceLinux() {
monitor_watcher_.StopWatchingFileDescriptor();
close(monitor_fd_);
@@ -110,26 +138,23 @@ void HidServiceLinux::Enumerate() {
ScopedUdevDevicePtr hid_dev(
udev_device_new_from_syspath(udev_.get(), udev_list_entry_get_name(i)));
if (hid_dev) {
- PlatformDeviceAdd(hid_dev.get());
+ PlatformAddDevice(hid_dev.get());
}
}
-
- initialized_ = true;
}
-void HidServiceLinux::PlatformDeviceAdd(udev_device* device) {
+void HidServiceLinux::PlatformAddDevice(udev_device* device) {
if (!device)
return;
- const char* device_id = udev_device_get_syspath(device);
- if (!device_id)
+ const char* device_path = udev_device_get_syspath(device);
+ if (!device_path)
return;
-
HidDeviceInfo device_info;
- device_info.device_id = device_id;
+ device_info.device_id = device_path;
- uint32 int_property = 0;
+ uint32_t int_property = 0;
const char* str_property = NULL;
const char* hid_id = udev_device_get_property_value(device, kHIDID);
@@ -161,7 +186,7 @@ void HidServiceLinux::PlatformDeviceAdd(udev_device* device) {
AddDevice(device_info);
}
-void HidServiceLinux::PlatformDeviceRemove(udev_device* raw_dev) {
+void HidServiceLinux::PlatformRemoveDevice(udev_device* raw_dev) {
// The returned the device is not referenced.
udev_device* hid_dev =
udev_device_get_parent_with_subsystem_devtype(raw_dev, "hid", NULL);
@@ -169,43 +194,12 @@ void HidServiceLinux::PlatformDeviceRemove(udev_device* raw_dev) {
if (!hid_dev)
return;
- const char* device_id = NULL;
- device_id = udev_device_get_syspath(hid_dev);
- if (device_id == NULL)
- return;
-
- RemoveDevice(device_id);
-}
-
-scoped_refptr<HidConnection> HidServiceLinux::Connect(std::string device_id) {
- if (!ContainsKey(devices_, device_id))
- return NULL;
- ScopedUdevDevicePtr hid_device(
- udev_device_new_from_syspath(udev_.get(), device_id.c_str()));
- if (hid_device) {
- scoped_refptr<HidConnectionLinux> connection =
- new HidConnectionLinux(devices_[device_id], hid_device.Pass());
- if (connection->initialized())
- return connection;
- }
- return NULL;
-}
-
-void HidServiceLinux::OnFileCanReadWithoutBlocking(int fd) {
- DCHECK_EQ(monitor_fd_, fd);
-
- ScopedUdevDevicePtr dev(udev_monitor_receive_device(monitor_.get()));
- if (!dev)
+ const char* device_path = NULL;
+ device_path = udev_device_get_syspath(hid_dev);
+ if (device_path == NULL)
return;
- std::string action(udev_device_get_action(dev.get()));
- if (action == kUdevActionAdd) {
- PlatformDeviceAdd(dev.get());
- } else if (action == kUdevActionRemove) {
- PlatformDeviceRemove(dev.get());
- }
+ RemoveDevice(device_path);
}
-void HidServiceLinux::OnFileCanWriteWithoutBlocking(int fd) {}
-
} // namespace dev