Reland of Parse USB interface association descriptors.

Reland of Parse USB interface association descriptors. (patchset #1 id:1 of https://codereview.chromium.org/1697863003/ )

Reason for revert:
Relanding this with a new implementation and fixed the test bug that led this issue to not be caught in the first place.

Original issue's description:
> Revert of Parse USB interface association descriptors. (patchset #3 id:100001 of https://codereview.chromium.org/1568673002/ )
>
> Reason for revert:
> http://crbug.com/586824
>
> Makes Chrome hang when using USB devices.
>
> Original issue's description:
> > Parse USB interface association descriptors.
> >
> > USB interface association descriptors are used to combine multiple
> > interfaces into a single functional group. This patch adds support for
> > parsing them out of the |extra_data| field left by libusb's parsing of
> > configuration, interface and endpoint descriptors. The resulting
> > association is then represented by setting the |first_interface| field
> > of each interface in a function to the |interface_number| of the first
> > interface in the function.
> >
> > WebUSB will use these associations to set permissions for an entire
> > function with a single descriptor.

BUG=492204, 586824
Committed: https://crrev.com/6a2b2b5dafce11505958fbb945088347540fb9f7
Cr-Commit-Position: refs/heads/master@{#383543}

[Reilly Grant (use Gerrit), 2016-02-16 18:35:29]

Created Reland of Parse USB interface association descriptors.

[Reilly Grant (use Gerrit), 2016-02-16 18:41:23]

Description was changed from

==========
Reland of Parse USB interface association descriptors. (patchset #1 id:1 of
https://codereview.chromium.org/1697863003/ )

Reason for revert:
Relanding this with a new implementation and fixed the test bug that led this
issue to not be caught in the first place.

Original issue's description:
> Revert of Parse USB interface association descriptors. (patchset #3 id:100001
of https://codereview.chromium.org/1568673002/ )
> 
> Reason for revert:
> http://crbug.com/586824
> 
> Makes Chrome hang when using USB devices.
> 
> Original issue's description:
> > Parse USB interface association descriptors.
> > 
> > USB interface association descriptors are used to combine multiple
> > interfaces into a single functional group. This patch adds support for
> > parsing them out of the |extra_data| field left by libusb's parsing of
> > configuration, interface and endpoint descriptors. The resulting
> > association is then represented by setting the |first_interface| field
> > of each interface in a function to the |interface_number| of the first
> > interface in the function.
> > 
> > WebUSB will use these associations to set permissions for an entire
> > function with a single descriptor.
> > 
> 
>
TBR=rockot@chromium.org,pfeldman@chromium.org,stevenjb@chromium.org,reillyg@c...
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=492204, 586824

TBR=rockot@chromium.org,pfeldman@chromium.org,stevenjb@chromium.org,gab@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=492204, 586824
==========

to

==========
Reland of Parse USB interface association descriptors. (patchset #1 id:1 of
https://codereview.chromium.org/1697863003/ )

Reason for revert:
Relanding this with a new implementation and fixed the test bug that led this
issue to not be caught in the first place.

Original issue's description:
> Revert of Parse USB interface association descriptors. (patchset #3 id:100001
of https://codereview.chromium.org/1568673002/ )
> 
> Reason for revert:
> http://crbug.com/586824
> 
> Makes Chrome hang when using USB devices.
> 
> Original issue's description:
> > Parse USB interface association descriptors.
> > 
> > USB interface association descriptors are used to combine multiple
> > interfaces into a single functional group. This patch adds support for
> > parsing them out of the |extra_data| field left by libusb's parsing of
> > configuration, interface and endpoint descriptors. The resulting
> > association is then represented by setting the |first_interface| field
> > of each interface in a function to the |interface_number| of the first
> > interface in the function.
> > 
> > WebUSB will use these associations to set permissions for an entire
> > function with a single descriptor.

BUG=492204, 586824
==========

[Reilly Grant (use Gerrit), 2016-02-16 18:41:31]

reillyg@chromium.org changed reviewers:
- gab@chromium.org, pfeldman@chromium.org, stevenjb@chromium.org

[Reilly Grant (use Gerrit), 2016-02-24 22:43:46]

Did I ever mail this out? I guess not? Please take a look.

[Ken Rockot(use gerrit already), 2016-03-25 19:25:21]

sorry. D:

lgtm

[Reilly Grant (use Gerrit), 2016-03-28 18:30:35]

The CQ bit was checked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2016-03-28 18:30:35]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/1707453002/#ps150001
(title: "Rebased.")

[commit-bot: I haz the power, 2016-03-28 18:30:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1707453002/150001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1707453002/150001

[commit-bot: I haz the power, 2016-03-28 19:45:26]

Description was changed from

==========
Reland of Parse USB interface association descriptors. (patchset #1 id:1 of
https://codereview.chromium.org/1697863003/ )

Reason for revert:
Relanding this with a new implementation and fixed the test bug that led this
issue to not be caught in the first place.

Original issue's description:
> Revert of Parse USB interface association descriptors. (patchset #3 id:100001
of https://codereview.chromium.org/1568673002/ )
> 
> Reason for revert:
> http://crbug.com/586824
> 
> Makes Chrome hang when using USB devices.
> 
> Original issue's description:
> > Parse USB interface association descriptors.
> > 
> > USB interface association descriptors are used to combine multiple
> > interfaces into a single functional group. This patch adds support for
> > parsing them out of the |extra_data| field left by libusb's parsing of
> > configuration, interface and endpoint descriptors. The resulting
> > association is then represented by setting the |first_interface| field
> > of each interface in a function to the |interface_number| of the first
> > interface in the function.
> > 
> > WebUSB will use these associations to set permissions for an entire
> > function with a single descriptor.

BUG=492204, 586824
==========

to

==========
Reland of Parse USB interface association descriptors. (patchset #1 id:1 of
https://codereview.chromium.org/1697863003/ )

Reason for revert:
Relanding this with a new implementation and fixed the test bug that led this
issue to not be caught in the first place.

Original issue's description:
> Revert of Parse USB interface association descriptors. (patchset #3 id:100001
of https://codereview.chromium.org/1568673002/ )
> 
> Reason for revert:
> http://crbug.com/586824
> 
> Makes Chrome hang when using USB devices.
> 
> Original issue's description:
> > Parse USB interface association descriptors.
> > 
> > USB interface association descriptors are used to combine multiple
> > interfaces into a single functional group. This patch adds support for
> > parsing them out of the |extra_data| field left by libusb's parsing of
> > configuration, interface and endpoint descriptors. The resulting
> > association is then represented by setting the |first_interface| field
> > of each interface in a function to the |interface_number| of the first
> > interface in the function.
> > 
> > WebUSB will use these associations to set permissions for an entire
> > function with a single descriptor.

BUG=492204, 586824
==========

[commit-bot: I haz the power, 2016-03-28 19:45:26]

Committed patchset #3 (id:150001)

[commit-bot: I haz the power, 2016-03-28 19:47:01]

Description was changed from

==========
Reland of Parse USB interface association descriptors. (patchset #1 id:1 of
https://codereview.chromium.org/1697863003/ )

Reason for revert:
Relanding this with a new implementation and fixed the test bug that led this
issue to not be caught in the first place.

Original issue's description:
> Revert of Parse USB interface association descriptors. (patchset #3 id:100001
of https://codereview.chromium.org/1568673002/ )
> 
> Reason for revert:
> http://crbug.com/586824
> 
> Makes Chrome hang when using USB devices.
> 
> Original issue's description:
> > Parse USB interface association descriptors.
> > 
> > USB interface association descriptors are used to combine multiple
> > interfaces into a single functional group. This patch adds support for
> > parsing them out of the |extra_data| field left by libusb's parsing of
> > configuration, interface and endpoint descriptors. The resulting
> > association is then represented by setting the |first_interface| field
> > of each interface in a function to the |interface_number| of the first
> > interface in the function.
> > 
> > WebUSB will use these associations to set permissions for an entire
> > function with a single descriptor.

BUG=492204, 586824
==========

to

==========
Reland of Parse USB interface association descriptors. (patchset #1 id:1 of
https://codereview.chromium.org/1697863003/ )

Reason for revert:
Relanding this with a new implementation and fixed the test bug that led this
issue to not be caught in the first place.

Original issue's description:
> Revert of Parse USB interface association descriptors. (patchset #3 id:100001
of https://codereview.chromium.org/1568673002/ )
>
> Reason for revert:
> http://crbug.com/586824
>
> Makes Chrome hang when using USB devices.
>
> Original issue's description:
> > Parse USB interface association descriptors.
> >
> > USB interface association descriptors are used to combine multiple
> > interfaces into a single functional group. This patch adds support for
> > parsing them out of the |extra_data| field left by libusb's parsing of
> > configuration, interface and endpoint descriptors. The resulting
> > association is then represented by setting the |first_interface| field
> > of each interface in a function to the |interface_number| of the first
> > interface in the function.
> >
> > WebUSB will use these associations to set permissions for an entire
> > function with a single descriptor.

BUG=492204, 586824

Committed: https://crrev.com/6a2b2b5dafce11505958fbb945088347540fb9f7
Cr-Commit-Position: refs/heads/master@{#383543}
==========

[commit-bot: I haz the power, 2016-03-28 19:47:02]

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