Add support for sending a USB SET_CONFIGURATION request.

Add support for sending a USB SET_CONFIGURATION request.

Some devices may have multiple configuration options, and Windows and
OS X do not always automatically select a device configuration when a
device is enumerated. This new function allows a Chrome App to select a
device configuration once the device is opened.

From a security and isolation perspective this function is no more
dangerous than chrome.usb.resetDevice.

BUG=443213
Committed: https://crrev.com/13cf2cfcb5325ecfff98ee8ccde723f22e606d22
Cr-Commit-Position: refs/heads/master@{#310591}

[Reilly Grant (use Gerrit), 2014-12-30 21:43:20]

Patchset #1 (id:1) has been deleted

[Reilly Grant (use Gerrit), 2014-12-30 22:00:32]

Patchset #1 (id:20001) has been deleted

[Reilly Grant (use Gerrit), 2014-12-30 22:09:23]

reillyg@chromium.org changed reviewers:
+ rockot@chromium.org

[Reilly Grant (use Gerrit), 2014-12-30 22:09:58]

reillyg@chromium.org changed reviewers:
+ vsevik@chromium.org

[Reilly Grant (use Gerrit), 2014-12-30 22:10:25]

reillyg@chromium.org changed reviewers:
+ jar@chromium.org

[Reilly Grant (use Gerrit), 2014-12-30 22:10:38]

Patchset #1 (id:40001) has been deleted

[Reilly Grant (use Gerrit), 2014-12-30 22:13:21]

rockot@, please take a look at this new API and the device code behind it.

vsevik@, please take a look at the necessary updates to the DevTools USB code.

jar@, please take a look at the new histogram value.

[Reilly Grant (use Gerrit), 2014-12-30 22:15:15]

Patchset #1 (id:60001) has been deleted

[jar (doing other things), 2014-12-30 22:31:27]

Singular addition to histograms.xml LGTM... but I haven't looked at any of the
other code.

[Reilly Grant (use Gerrit), 2015-01-05 18:17:03]

reillyg@chromium.org changed reviewers:
+ meacer@chromium.org

[Reilly Grant (use Gerrit), 2015-01-05 18:17:24]

Adding Mustafa for security review.

[dgozman, 2015-01-06 10:17:27]

dgozman@chromium.org changed reviewers:
+ dgozman@chromium.org

[dgozman, 2015-01-06 10:17:28]

devtools lgtm

btw, do you plan on making GetConfiguration call work on windows with already
opened handle?

https://codereview.chromium.org/826283002/diff/80001/chrome/browser/devtools/...
File chrome/browser/devtools/device/usb/android_usb_device.cc (right):

https://codereview.chromium.org/826283002/diff/80001/chrome/browser/devtools/...
chrome/browser/devtools/device/usb/android_usb_device.cc:261: if (!config) {
nit: we don't usually wrap one-line statements with braces.

[Reilly Grant (use Gerrit), 2015-01-06 19:14:31]

On 2015/01/06 10:17:28, dgozman wrote:
> devtools lgtm
> 
> btw, do you plan on making GetConfiguration call work on windows with already
> opened handle?

It should work as is. GetConfiguration returns nullptr if the device is in an
unconfigured state. Calling SetConfiguration refreshes the cached configuration
information.

[Ken Rockot(use gerrit already), 2015-01-06 23:51:37]

lgtm

[Reilly Grant (use Gerrit), 2015-01-07 00:16:45]

Patchset #2 (id:100001) has been deleted

[meacer, 2015-01-08 14:47:15]

Lgtm

[Reilly Grant (use Gerrit), 2015-01-08 16:05:10]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2015-01-08 16:05:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/826283002/120001

[commit-bot: I haz the power, 2015-01-08 16:12:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-01-08 16:12:43]

Try jobs failed on following builders:
  android_compile_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_compile...)
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Reilly Grant (use Gerrit), 2015-01-08 19:08:07]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2015-01-08 19:09:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/826283002/140001

[commit-bot: I haz the power, 2015-01-08 20:47:20]

Committed patchset #3 (id:140001)

[commit-bot: I haz the power, 2015-01-08 20:48:53]

Patchset 3 (id:??) landed as
https://crrev.com/13cf2cfcb5325ecfff98ee8ccde723f22e606d22
Cr-Commit-Position: refs/heads/master@{#310591}