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

Unified Diff: chrome/browser/devtools/device/usb/android_usb_device.cc

Issue 567003002: Revert of Convert device::UsbConfigDescriptor and friends to structs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/devtools/device/usb/android_usb_browsertest.cc ('k') | device/usb/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/devtools/device/usb/android_usb_device.cc
diff --git a/chrome/browser/devtools/device/usb/android_usb_device.cc b/chrome/browser/devtools/device/usb/android_usb_device.cc
index 55b47c7bf3a734641bb3e2cec9e2c438d53f5848..8ef9416ff8ca1b28054c3e241325e3ed1fd9b45e 100644
--- a/chrome/browser/devtools/device/usb/android_usb_device.cc
+++ b/chrome/browser/devtools/device/usb/android_usb_device.cc
@@ -19,8 +19,8 @@
#include "content/public/browser/browser_thread.h"
#include "crypto/rsa_private_key.h"
#include "device/core/device_client.h"
-#include "device/usb/usb_descriptors.h"
#include "device/usb/usb_device.h"
+#include "device/usb/usb_interface.h"
#include "device/usb/usb_service.h"
#include "net/base/ip_endpoint.h"
#include "net/base/net_errors.h"
@@ -29,6 +29,7 @@
using device::UsbConfigDescriptor;
using device::UsbDevice;
using device::UsbDeviceHandle;
+using device::UsbInterfaceAltSettingDescriptor;
using device::UsbInterfaceDescriptor;
using device::UsbEndpointDescriptor;
using device::UsbService;
@@ -58,12 +59,17 @@
base::LazyInstance<std::vector<AndroidUsbDevice*> >::Leaky g_devices =
LAZY_INSTANCE_INITIALIZER;
-bool IsAndroidInterface(const UsbInterfaceDescriptor& interface) {
- if (interface.alternate_setting != 0 ||
- interface.interface_class != kAdbClass ||
- interface.interface_subclass != kAdbSubclass ||
- interface.interface_protocol != kAdbProtocol ||
- interface.endpoints.size() != 2) {
+bool IsAndroidInterface(scoped_refptr<const UsbInterfaceDescriptor> interface) {
+ if (interface->GetNumAltSettings() == 0)
+ return false;
+
+ scoped_refptr<const UsbInterfaceAltSettingDescriptor> idesc =
+ interface->GetAltSetting(0);
+
+ if (idesc->GetInterfaceClass() != kAdbClass ||
+ idesc->GetInterfaceSubclass() != kAdbSubclass ||
+ idesc->GetInterfaceProtocol() != kAdbProtocol ||
+ idesc->GetNumEndpoints() != 2) {
return false;
}
return true;
@@ -72,40 +78,40 @@
scoped_refptr<AndroidUsbDevice> ClaimInterface(
crypto::RSAPrivateKey* rsa_key,
scoped_refptr<UsbDeviceHandle> usb_handle,
- const UsbInterfaceDescriptor& interface) {
+ scoped_refptr<const UsbInterfaceDescriptor> interface,
+ int interface_id) {
+ scoped_refptr<const UsbInterfaceAltSettingDescriptor> idesc =
+ interface->GetAltSetting(0);
+
int inbound_address = 0;
int outbound_address = 0;
int zero_mask = 0;
- for (UsbEndpointDescriptor::Iterator endpointIt = interface.endpoints.begin();
- endpointIt != interface.endpoints.end();
- ++endpointIt) {
- if (endpointIt->transfer_type != device::USB_TRANSFER_BULK)
+ for (size_t i = 0; i < idesc->GetNumEndpoints(); ++i) {
+ scoped_refptr<const UsbEndpointDescriptor> edesc =
+ idesc->GetEndpoint(i);
+ if (edesc->GetTransferType() != device::USB_TRANSFER_BULK)
continue;
- if (endpointIt->direction == device::USB_DIRECTION_INBOUND)
- inbound_address = endpointIt->address;
+ if (edesc->GetDirection() == device::USB_DIRECTION_INBOUND)
+ inbound_address = edesc->GetAddress();
else
- outbound_address = endpointIt->address;
- zero_mask = endpointIt->maximum_packet_size - 1;
+ outbound_address = edesc->GetAddress();
+ zero_mask = edesc->GetMaximumPacketSize() - 1;
}
if (inbound_address == 0 || outbound_address == 0)
return NULL;
- if (!usb_handle->ClaimInterface(interface.interface_number))
+ if (!usb_handle->ClaimInterface(interface_id))
return NULL;
base::string16 serial;
if (!usb_handle->GetSerial(&serial) || serial.empty())
return NULL;
- return new AndroidUsbDevice(rsa_key,
- usb_handle,
- base::UTF16ToASCII(serial),
- inbound_address,
- outbound_address,
- zero_mask,
- interface.interface_number);
+ return new AndroidUsbDevice(rsa_key, usb_handle, base::UTF16ToASCII(serial),
+ inbound_address, outbound_address, zero_mask,
+ interface_id);
}
uint32 Checksum(const std::string& data) {
@@ -201,11 +207,12 @@
bool success) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
if (success) {
- const UsbConfigDescriptor& config = device->GetConfiguration();
+ scoped_refptr<UsbConfigDescriptor> config = device->ListInterfaces();
scoped_refptr<UsbDeviceHandle> usb_handle = device->Open();
if (usb_handle.get()) {
scoped_refptr<AndroidUsbDevice> android_device =
- ClaimInterface(rsa_key, usb_handle, config.interfaces[interface_id]);
+ ClaimInterface(rsa_key, usb_handle, config->GetInterface(interface_id),
+ interface_id);
if (android_device.get())
devices->push_back(android_device.get());
else
@@ -222,17 +229,15 @@
if (service != NULL)
service->GetDevices(&usb_devices);
int device_count = 0;
- for (UsbDevices::iterator deviceIt = usb_devices.begin();
- deviceIt != usb_devices.end();
- ++deviceIt) {
- const UsbConfigDescriptor& config = (*deviceIt)->GetConfiguration();
-
- for (UsbInterfaceDescriptor::Iterator ifaceIt = config.interfaces.begin();
- ifaceIt != config.interfaces.end();
- ++ifaceIt) {
- if (IsAndroidInterface(*ifaceIt)) {
+ for (UsbDevices::iterator it = usb_devices.begin(); it != usb_devices.end();
+ ++it) {
+ scoped_refptr<UsbConfigDescriptor> config = (*it)->ListInterfaces();
+ if (!config.get())
+ continue;
+
+ for (size_t j = 0; j < config->GetNumInterfaces(); ++j) {
+ if (IsAndroidInterface(config->GetInterface(j)))
++device_count;
- }
}
}
return device_count;
@@ -259,13 +264,16 @@
for (UsbDevices::iterator it = usb_devices.begin(); it != usb_devices.end();
++it) {
- const UsbConfigDescriptor& config = (*it)->GetConfiguration();
+ scoped_refptr<UsbConfigDescriptor> config = (*it)->ListInterfaces();
+ if (!config.get()) {
+ barrier.Run();
+ continue;
+ }
bool has_android_interface = false;
- for (size_t j = 0; j < config.interfaces.size(); ++j) {
- if (!IsAndroidInterface(config.interfaces[j])) {
+ for (size_t j = 0; j < config->GetNumInterfaces(); ++j) {
+ if (!IsAndroidInterface(config->GetInterface(j)))
continue;
- }
// Request permission on Chrome OS.
#if defined(OS_CHROMEOS)
« no previous file with comments | « chrome/browser/devtools/device/usb/android_usb_browsertest.cc ('k') | device/usb/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698