Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(426)

Side by Side Diff: extensions/browser/api/hid/hid_device_manager.cc

Issue 514923002: Add more generic filters to the chrome.hid.getDevices API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clean up the chrome.hid documentation even more. Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « extensions/browser/api/hid/hid_device_manager.h ('k') | extensions/common/api/hid.idl » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "extensions/browser/api/hid/hid_device_manager.h" 5 #include "extensions/browser/api/hid/hid_device_manager.h"
6 6
7 #include <limits> 7 #include <limits>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/lazy_instance.h" 10 #include "base/lazy_instance.h"
11 #include "device/hid/hid_device_filter.h"
11 #include "device/hid/hid_service.h" 12 #include "device/hid/hid_service.h"
12 #include "extensions/browser/api/extensions_api_client.h" 13 #include "extensions/browser/api/extensions_api_client.h"
14 #include "extensions/common/permissions/permissions_data.h"
15 #include "extensions/common/permissions/usb_device_permission.h"
13 16
17 using device::HidDeviceFilter;
14 using device::HidService; 18 using device::HidService;
15 using device::HidUsageAndPage; 19 using device::HidUsageAndPage;
16 20
17 namespace extensions { 21 namespace extensions {
18 22
19 HidDeviceManager::HidDeviceManager(content::BrowserContext* context) 23 HidDeviceManager::HidDeviceManager(content::BrowserContext* context)
20 : next_resource_id_(0) {} 24 : next_resource_id_(0) {}
21 25
22 HidDeviceManager::~HidDeviceManager() {} 26 HidDeviceManager::~HidDeviceManager() {}
23 27
24 // static 28 // static
25 BrowserContextKeyedAPIFactory<HidDeviceManager>* 29 BrowserContextKeyedAPIFactory<HidDeviceManager>*
26 HidDeviceManager::GetFactoryInstance() { 30 HidDeviceManager::GetFactoryInstance() {
27 static base::LazyInstance<BrowserContextKeyedAPIFactory<HidDeviceManager> > 31 static base::LazyInstance<BrowserContextKeyedAPIFactory<HidDeviceManager> >
28 factory = LAZY_INSTANCE_INITIALIZER; 32 factory = LAZY_INSTANCE_INITIALIZER;
29 return &factory.Get(); 33 return &factory.Get();
30 } 34 }
31 35
32 scoped_ptr<base::ListValue> HidDeviceManager::GetApiDevices( 36 scoped_ptr<base::ListValue> HidDeviceManager::GetApiDevices(
33 uint16_t vendor_id, 37 const Extension* extension,
34 uint16_t product_id) { 38 const std::vector<HidDeviceFilter>& filters) {
35 UpdateDevices(); 39 UpdateDevices();
36 40
37 HidService* hid_service = ExtensionsAPIClient::Get()->GetHidService(); 41 HidService* hid_service = ExtensionsAPIClient::Get()->GetHidService();
38 DCHECK(hid_service); 42 DCHECK(hid_service);
39 base::ListValue* api_devices = new base::ListValue(); 43 base::ListValue* api_devices = new base::ListValue();
40 for (ResourceIdToDeviceIdMap::const_iterator device_iter = 44 for (ResourceIdToDeviceIdMap::const_iterator device_iter =
41 device_ids_.begin(); 45 device_ids_.begin();
42 device_iter != device_ids_.end(); 46 device_iter != device_ids_.end();
43 ++device_iter) { 47 ++device_iter) {
44 int resource_id = device_iter->first; 48 int resource_id = device_iter->first;
45 device::HidDeviceId device_id = device_iter->second; 49 device::HidDeviceId device_id = device_iter->second;
46 device::HidDeviceInfo device_info; 50 device::HidDeviceInfo device_info;
47 51
48 if (hid_service->GetDeviceInfo(device_id, &device_info)) { 52 if (hid_service->GetDeviceInfo(device_id, &device_info)) {
49 if (device_info.vendor_id == vendor_id && 53 if (!filters.empty() &&
50 device_info.product_id == product_id) { 54 !HidDeviceFilter::MatchesAny(device_info, filters)) {
51 core_api::hid::HidDeviceInfo api_device_info; 55 continue;
52 api_device_info.device_id = resource_id; 56 }
53 api_device_info.vendor_id = device_info.vendor_id;
54 api_device_info.product_id = device_info.product_id;
55 api_device_info.max_input_report_size =
56 device_info.max_input_report_size;
57 api_device_info.max_output_report_size =
58 device_info.max_output_report_size;
59 api_device_info.max_feature_report_size =
60 device_info.max_feature_report_size;
61 57
62 for (std::vector<device::HidCollectionInfo>::const_iterator 58 if (!HasPermission(extension, device_info)) {
63 collections_iter = device_info.collections.begin(); 59 continue;
64 collections_iter != device_info.collections.end(); 60 }
65 ++collections_iter) {
66 device::HidCollectionInfo collection = *collections_iter;
67 61
68 // Don't expose sensitive data. 62 core_api::hid::HidDeviceInfo api_device_info;
69 if (collection.usage.IsProtected()) { 63 api_device_info.device_id = resource_id;
70 continue; 64 api_device_info.vendor_id = device_info.vendor_id;
71 } 65 api_device_info.product_id = device_info.product_id;
66 api_device_info.max_input_report_size = device_info.max_input_report_size;
67 api_device_info.max_output_report_size =
68 device_info.max_output_report_size;
69 api_device_info.max_feature_report_size =
70 device_info.max_feature_report_size;
72 71
73 core_api::hid::HidCollectionInfo* api_collection = 72 for (std::vector<device::HidCollectionInfo>::const_iterator
74 new core_api::hid::HidCollectionInfo(); 73 collections_iter = device_info.collections.begin();
75 api_collection->usage_page = collection.usage.usage_page; 74 collections_iter != device_info.collections.end();
76 api_collection->usage = collection.usage.usage; 75 ++collections_iter) {
76 const device::HidCollectionInfo& collection = *collections_iter;
77 77
78 api_collection->report_ids.resize(collection.report_ids.size()); 78 // Don't expose sensitive data.
79 std::copy(collection.report_ids.begin(), 79 if (collection.usage.IsProtected()) {
80 collection.report_ids.end(), 80 continue;
81 api_collection->report_ids.begin());
82
83 api_device_info.collections.push_back(
84 make_linked_ptr(api_collection));
85 } 81 }
86 82
87 // Expose devices with which user can communicate. 83 core_api::hid::HidCollectionInfo* api_collection =
88 if (api_device_info.collections.size() > 0) 84 new core_api::hid::HidCollectionInfo();
89 api_devices->Append(api_device_info.ToValue().release()); 85 api_collection->usage_page = collection.usage.usage_page;
86 api_collection->usage = collection.usage.usage;
87
88 api_collection->report_ids.resize(collection.report_ids.size());
89 std::copy(collection.report_ids.begin(),
90 collection.report_ids.end(),
91 api_collection->report_ids.begin());
92
93 api_device_info.collections.push_back(make_linked_ptr(api_collection));
94 }
95
96 // Expose devices with which user can communicate.
97 if (api_device_info.collections.size() > 0) {
98 api_devices->Append(api_device_info.ToValue().release());
90 } 99 }
91 } 100 }
92 } 101 }
93 102
94 return scoped_ptr<base::ListValue>(api_devices); 103 return scoped_ptr<base::ListValue>(api_devices);
95 } 104 }
96 105
97 bool HidDeviceManager::GetDeviceInfo(int resource_id, 106 bool HidDeviceManager::GetDeviceInfo(int resource_id,
98 device::HidDeviceInfo* device_info) { 107 device::HidDeviceInfo* device_info) {
99 UpdateDevices(); 108 UpdateDevices();
100 HidService* hid_service = ExtensionsAPIClient::Get()->GetHidService(); 109 HidService* hid_service = ExtensionsAPIClient::Get()->GetHidService();
101 DCHECK(hid_service); 110 DCHECK(hid_service);
102 111
103 ResourceIdToDeviceIdMap::const_iterator device_iter = 112 ResourceIdToDeviceIdMap::const_iterator device_iter =
104 device_ids_.find(resource_id); 113 device_ids_.find(resource_id);
105 if (device_iter == device_ids_.end()) 114 if (device_iter == device_ids_.end())
106 return false; 115 return false;
107 116
108 return hid_service->GetDeviceInfo(device_iter->second, device_info); 117 return hid_service->GetDeviceInfo(device_iter->second, device_info);
109 } 118 }
110 119
120 bool HidDeviceManager::HasPermission(const Extension* extension,
121 const device::HidDeviceInfo& device_info) {
122 UsbDevicePermission::CheckParam usbParam(
123 device_info.vendor_id,
124 device_info.product_id,
125 UsbDevicePermissionData::UNSPECIFIED_INTERFACE);
126 if (extension->permissions_data()->CheckAPIPermissionWithParam(
127 APIPermission::kUsbDevice, &usbParam)) {
128 return true;
129 }
130
131 return false;
132 }
133
111 void HidDeviceManager::UpdateDevices() { 134 void HidDeviceManager::UpdateDevices() {
112 thread_checker_.CalledOnValidThread(); 135 thread_checker_.CalledOnValidThread();
113 HidService* hid_service = ExtensionsAPIClient::Get()->GetHidService(); 136 HidService* hid_service = ExtensionsAPIClient::Get()->GetHidService();
114 DCHECK(hid_service); 137 DCHECK(hid_service);
115 138
116 std::vector<device::HidDeviceInfo> devices; 139 std::vector<device::HidDeviceInfo> devices;
117 hid_service->GetDevices(&devices); 140 hid_service->GetDevices(&devices);
118 141
119 // Build an updated bidi mapping between resource ID and underlying device ID. 142 // Build an updated bidi mapping between resource ID and underlying device ID.
120 DeviceIdToResourceIdMap new_resource_ids; 143 DeviceIdToResourceIdMap new_resource_ids;
(...skipping 13 matching lines...) Expand all
134 new_id = next_resource_id_++; 157 new_id = next_resource_id_++;
135 } 158 }
136 new_resource_ids[device_info.device_id] = new_id; 159 new_resource_ids[device_info.device_id] = new_id;
137 new_device_ids[new_id] = device_info.device_id; 160 new_device_ids[new_id] = device_info.device_id;
138 } 161 }
139 device_ids_.swap(new_device_ids); 162 device_ids_.swap(new_device_ids);
140 resource_ids_.swap(new_resource_ids); 163 resource_ids_.swap(new_resource_ids);
141 } 164 }
142 165
143 } // namespace extensions 166 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/browser/api/hid/hid_device_manager.h ('k') | extensions/common/api/hid.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698