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

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

Issue 369923002: Revert 281133 "chrome.hid: enrich model with report IDs" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 5 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 | Annotate | Revision Log
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 "chrome/browser/extensions/api/hid/hid_device_manager.h" 5 #include "chrome/browser/extensions/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"
(...skipping 25 matching lines...) Expand all
36 HidService* hid_service = HidService::GetInstance(); 36 HidService* hid_service = HidService::GetInstance();
37 DCHECK(hid_service); 37 DCHECK(hid_service);
38 base::ListValue* api_devices = new base::ListValue(); 38 base::ListValue* api_devices = new base::ListValue();
39 for (ResourceIdToDeviceIdMap::const_iterator device_iter = 39 for (ResourceIdToDeviceIdMap::const_iterator device_iter =
40 device_ids_.begin(); 40 device_ids_.begin();
41 device_iter != device_ids_.end(); 41 device_iter != device_ids_.end();
42 ++device_iter) { 42 ++device_iter) {
43 int resource_id = device_iter->first; 43 int resource_id = device_iter->first;
44 device::HidDeviceId device_id = device_iter->second; 44 device::HidDeviceId device_id = device_iter->second;
45 device::HidDeviceInfo device_info; 45 device::HidDeviceInfo device_info;
46
47 if (hid_service->GetDeviceInfo(device_id, &device_info)) { 46 if (hid_service->GetDeviceInfo(device_id, &device_info)) {
48 if (device_info.vendor_id == vendor_id && 47 if (device_info.vendor_id == vendor_id &&
49 device_info.product_id == product_id) { 48 device_info.product_id == product_id &&
49 IsDeviceAccessible(device_info)) {
50 api::hid::HidDeviceInfo api_device_info; 50 api::hid::HidDeviceInfo api_device_info;
51 api_device_info.device_id = resource_id; 51 api_device_info.device_id = resource_id;
52 api_device_info.vendor_id = device_info.vendor_id; 52 api_device_info.vendor_id = device_info.vendor_id;
53 api_device_info.product_id = device_info.product_id; 53 api_device_info.product_id = device_info.product_id;
54 api_device_info.max_input_report_size = 54 for (std::vector<device::HidUsageAndPage>::const_iterator usage_iter =
55 device_info.max_input_report_size; 55 device_info.usages.begin();
56 api_device_info.max_output_report_size = 56 usage_iter != device_info.usages.end();
57 device_info.max_output_report_size; 57 ++usage_iter) {
58 api_device_info.max_feature_report_size = 58 api::hid::HidUsageAndPage* usage_and_page =
59 device_info.max_feature_report_size; 59 new api::hid::HidUsageAndPage();
60 60 usage_and_page->usage_page = (*usage_iter).usage_page;
61 for (std::vector<device::HidCollectionInfo>::const_iterator 61 usage_and_page->usage = (*usage_iter).usage;
62 collections_iter = device_info.collections.begin(); 62 linked_ptr<api::hid::HidUsageAndPage> usage_and_page_ptr(
63 collections_iter != device_info.collections.end(); 63 usage_and_page);
64 ++collections_iter) { 64 api_device_info.usages.push_back(usage_and_page_ptr);
65 device::HidCollectionInfo collection = *collections_iter;
66
67 // Don't expose sensitive data.
68 if (collection.usage.IsProtected()) {
69 continue;
70 }
71
72 api::hid::HidCollectionInfo* api_collection =
73 new api::hid::HidCollectionInfo();
74 api_collection->usage_page = collection.usage.usage_page;
75 api_collection->usage = collection.usage.usage;
76
77 api_collection->report_ids.resize(collection.report_ids.size());
78 std::copy(collection.report_ids.begin(),
79 collection.report_ids.end(),
80 api_collection->report_ids.begin());
81
82 api_device_info.collections.push_back(
83 make_linked_ptr(api_collection));
84 } 65 }
85 66 api_devices->Append(api_device_info.ToValue().release());
86 // Expose devices with which user can communicate.
87 if (api_device_info.collections.size() > 0)
88 api_devices->Append(api_device_info.ToValue().release());
89 } 67 }
90 } 68 }
91 } 69 }
92
93 return scoped_ptr<base::ListValue>(api_devices); 70 return scoped_ptr<base::ListValue>(api_devices);
94 } 71 }
95 72
96 bool HidDeviceManager::GetDeviceInfo(int resource_id, 73 bool HidDeviceManager::GetDeviceInfo(int resource_id,
97 device::HidDeviceInfo* device_info) { 74 device::HidDeviceInfo* device_info) {
98 UpdateDevices(); 75 UpdateDevices();
99 HidService* hid_service = HidService::GetInstance(); 76 HidService* hid_service = HidService::GetInstance();
100 DCHECK(hid_service); 77 DCHECK(hid_service);
101 78
102 ResourceIdToDeviceIdMap::const_iterator device_iter = 79 ResourceIdToDeviceIdMap::const_iterator device_iter =
(...skipping 29 matching lines...) Expand all
132 DCHECK_LT(next_resource_id_, std::numeric_limits<int>::max()); 109 DCHECK_LT(next_resource_id_, std::numeric_limits<int>::max());
133 new_id = next_resource_id_++; 110 new_id = next_resource_id_++;
134 } 111 }
135 new_resource_ids[device_info.device_id] = new_id; 112 new_resource_ids[device_info.device_id] = new_id;
136 new_device_ids[new_id] = device_info.device_id; 113 new_device_ids[new_id] = device_info.device_id;
137 } 114 }
138 device_ids_.swap(new_device_ids); 115 device_ids_.swap(new_device_ids);
139 resource_ids_.swap(new_resource_ids); 116 resource_ids_.swap(new_resource_ids);
140 } 117 }
141 118
119 // static
120 // TODO(rockot): Add some tests for this.
121 bool HidDeviceManager::IsDeviceAccessible(
122 const device::HidDeviceInfo& device_info) {
123 for (std::vector<device::HidUsageAndPage>::const_iterator iter =
124 device_info.usages.begin();
125 iter != device_info.usages.end(); ++iter) {
126 if (!IsUsageAccessible(*iter)) {
127 return false;
128 }
129 }
130 return true;
131 }
132
133 // static
134 bool HidDeviceManager::IsUsageAccessible(
135 const HidUsageAndPage& usage_and_page) {
136 if (usage_and_page.usage_page == HidUsageAndPage::kPageKeyboard)
137 return false;
138
139 if (usage_and_page.usage_page != HidUsageAndPage::kPageGenericDesktop)
140 return true;
141
142 uint16_t usage = usage_and_page.usage;
143 if (usage == HidUsageAndPage::kGenericDesktopPointer ||
144 usage == HidUsageAndPage::kGenericDesktopMouse ||
145 usage == HidUsageAndPage::kGenericDesktopKeyboard ||
146 usage == HidUsageAndPage::kGenericDesktopKeypad) {
147 return false;
148 }
149
150 if (usage >= HidUsageAndPage::kGenericDesktopSystemControl &&
151 usage <= HidUsageAndPage::kGenericDesktopSystemWarmRestart) {
152 return false;
153 }
154
155 if (usage >= HidUsageAndPage::kGenericDesktopSystemDock &&
156 usage <= HidUsageAndPage::kGenericDesktopSystemDisplaySwap) {
157 return false;
158 }
159
160 return true;
161 }
162
142 } // namespace extensions 163 } // namespace extensions
OLDNEW
« no previous file with comments | « trunk/src/chrome/browser/extensions/api/hid/hid_device_manager.h ('k') | trunk/src/chrome/common/extensions/api/hid.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698