Reland of Update device/usb and its Mojo interface for variable size ISO packets.

Reland of Update device/usb and its Mojo interface for variable size ISO packets. (patchset #1 id:1 of https://codereview.chromium.org/1658953003/ )

Reason for revert:
Fixed the uninitialized test buffer.

Original issue's description:
> Revert of Update device/usb and its Mojo interface for variable size ISO packets. (patchset #3 id:120001 of https://codereview.chromium.org/1618393004/ )
>
> Reason for revert:
> Uninit error from UsbApiTest.TransferFailure
>
> BUG=583343
>
> Original issue's description:
> > Update device/usb and its Mojo interface for variable size ISO packets.
> >
> > To support the WebUSB API our underlying USB library needs to support
> > isochronous transfers with full control over the packet size. We also
> > need to know the completion status of each packet which was previously
> > not available.
> >
> > This patch updates the interface to match that provided by libusb and
> > the underlying platform specific APIs.
> >
> > BUG=492204
> >
> > Committed: https://crrev.com/051c98e9d3e843295d659b5676fcfa9dc1be5da6
> > Cr-Commit-Position: refs/heads/master@{#372844}
>
> TBR=scheib@chromium.org,dgozman@chromium.org,reillyg@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=492204
>
> Committed: https://crrev.com/c40c36ef3d3a722e0df920a4bbae25695f0b18e8
> Cr-Commit-Position: refs/heads/master@{#372971}

TBR=scheib@chromium.org,dgozman@chromium.org,zhaoqin@chromium.org
BUG=583343
Committed: https://crrev.com/ccbaed85d9538dc645535efdb58c0af6ebafdd43
Cr-Commit-Position: refs/heads/master@{#373065}

[Reilly Grant (use Gerrit), 2016-02-02 21:12:21]

Created Reland of Update device/usb and its Mojo interface for variable size ISO
packets.

[Reilly Grant (use Gerrit), 2016-02-02 21:13:46]

Patchset #1 (id:1) has been deleted

[Reilly Grant (use Gerrit), 2016-02-02 21:14:17]

https://codereview.chromium.org/1665453002/diff/270001/extensions/browser/api...
File extensions/browser/api/usb/usb_apitest.cc (right):

https://codereview.chromium.org/1665453002/diff/270001/extensions/browser/api...
extensions/browser/api/usb/usb_apitest.cc:75: memset(io_buffer->data(), 0,
total_length);  // Avoid uninitialized reads.
For the record this is the line I had to add in this reland.

[Reilly Grant (use Gerrit), 2016-02-02 21:14:21]

The CQ bit was checked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2016-02-02 21:15:05]

Description was changed from

==========
Reland of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #1 id:1 of https://codereview.chromium.org/1658953003/ )

Reason for revert:
Fixed the uninitialized test buffer.

Original issue's description:
> Revert of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #3 id:120001 of https://codereview.chromium.org/1618393004/ )
> 
> Reason for revert:
> Uninit error from UsbApiTest.TransferFailure
> 
> BUG=583343
> 
> Original issue's description:
> > Update device/usb and its Mojo interface for variable size ISO packets.
> >
> > To support the WebUSB API our underlying USB library needs to support
> > isochronous transfers with full control over the packet size. We also
> > need to know the completion status of each packet which was previously
> > not available.
> >
> > This patch updates the interface to match that provided by libusb and
> > the underlying platform specific APIs.
> >
> > BUG=492204
> >
> > Committed: https://crrev.com/051c98e9d3e843295d659b5676fcfa9dc1be5da6
> > Cr-Commit-Position: refs/heads/master@{#372844}
> 
> TBR=scheib@chromium.org,dgozman@chromium.org,reillyg@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=492204
> 
> Committed: https://crrev.com/c40c36ef3d3a722e0df920a4bbae25695f0b18e8
> Cr-Commit-Position: refs/heads/master@{#372971}

TBR=scheib@chromium.org,dgozman@chromium.org,zhaoqin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=583343
==========

to

==========
Reland of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #1 id:1 of https://codereview.chromium.org/1658953003/ )

Reason for revert:
Fixed the uninitialized test buffer.

Original issue's description:
> Revert of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #3 id:120001 of https://codereview.chromium.org/1618393004/ )
> 
> Reason for revert:
> Uninit error from UsbApiTest.TransferFailure
> 
> BUG=583343
> 
> Original issue's description:
> > Update device/usb and its Mojo interface for variable size ISO packets.
> >
> > To support the WebUSB API our underlying USB library needs to support
> > isochronous transfers with full control over the packet size. We also
> > need to know the completion status of each packet which was previously
> > not available.
> >
> > This patch updates the interface to match that provided by libusb and
> > the underlying platform specific APIs.
> >
> > BUG=492204
> >
> > Committed: https://crrev.com/051c98e9d3e843295d659b5676fcfa9dc1be5da6
> > Cr-Commit-Position: refs/heads/master@{#372844}
> 
> TBR=scheib@chromium.org,dgozman@chromium.org,reillyg@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=492204
> 
> Committed: https://crrev.com/c40c36ef3d3a722e0df920a4bbae25695f0b18e8
> Cr-Commit-Position: refs/heads/master@{#372971}

TBR=scheib@chromium.org,dgozman@chromium.org,zhaoqin@chromium.org
BUG=583343
==========

[commit-bot: I haz the power, 2016-02-02 21:16:29]

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

[scheib, 2016-02-02 21:16:59]

Would you re-upload to previous patch so that patch diffs work (fore reviewers
and posterity), please?

[scheib, 2016-02-02 21:21:17]

[offline conversation happened] OK, LGTM for this one, easier review next time.

[commit-bot: I haz the power, 2016-02-02 22:50:40]

Description was changed from

==========
Reland of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #1 id:1 of https://codereview.chromium.org/1658953003/ )

Reason for revert:
Fixed the uninitialized test buffer.

Original issue's description:
> Revert of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #3 id:120001 of https://codereview.chromium.org/1618393004/ )
> 
> Reason for revert:
> Uninit error from UsbApiTest.TransferFailure
> 
> BUG=583343
> 
> Original issue's description:
> > Update device/usb and its Mojo interface for variable size ISO packets.
> >
> > To support the WebUSB API our underlying USB library needs to support
> > isochronous transfers with full control over the packet size. We also
> > need to know the completion status of each packet which was previously
> > not available.
> >
> > This patch updates the interface to match that provided by libusb and
> > the underlying platform specific APIs.
> >
> > BUG=492204
> >
> > Committed: https://crrev.com/051c98e9d3e843295d659b5676fcfa9dc1be5da6
> > Cr-Commit-Position: refs/heads/master@{#372844}
> 
> TBR=scheib@chromium.org,dgozman@chromium.org,reillyg@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=492204
> 
> Committed: https://crrev.com/c40c36ef3d3a722e0df920a4bbae25695f0b18e8
> Cr-Commit-Position: refs/heads/master@{#372971}

TBR=scheib@chromium.org,dgozman@chromium.org,zhaoqin@chromium.org
BUG=583343
==========

to

==========
Reland of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #1 id:1 of https://codereview.chromium.org/1658953003/ )

Reason for revert:
Fixed the uninitialized test buffer.

Original issue's description:
> Revert of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #3 id:120001 of https://codereview.chromium.org/1618393004/ )
> 
> Reason for revert:
> Uninit error from UsbApiTest.TransferFailure
> 
> BUG=583343
> 
> Original issue's description:
> > Update device/usb and its Mojo interface for variable size ISO packets.
> >
> > To support the WebUSB API our underlying USB library needs to support
> > isochronous transfers with full control over the packet size. We also
> > need to know the completion status of each packet which was previously
> > not available.
> >
> > This patch updates the interface to match that provided by libusb and
> > the underlying platform specific APIs.
> >
> > BUG=492204
> >
> > Committed: https://crrev.com/051c98e9d3e843295d659b5676fcfa9dc1be5da6
> > Cr-Commit-Position: refs/heads/master@{#372844}
> 
> TBR=scheib@chromium.org,dgozman@chromium.org,reillyg@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=492204
> 
> Committed: https://crrev.com/c40c36ef3d3a722e0df920a4bbae25695f0b18e8
> Cr-Commit-Position: refs/heads/master@{#372971}

TBR=scheib@chromium.org,dgozman@chromium.org,zhaoqin@chromium.org
BUG=583343
==========

[commit-bot: I haz the power, 2016-02-02 22:50:41]

Committed patchset #1 (id:270001)

[commit-bot: I haz the power, 2016-02-02 22:51:54]

Description was changed from

==========
Reland of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #1 id:1 of https://codereview.chromium.org/1658953003/ )

Reason for revert:
Fixed the uninitialized test buffer.

Original issue's description:
> Revert of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #3 id:120001 of https://codereview.chromium.org/1618393004/ )
> 
> Reason for revert:
> Uninit error from UsbApiTest.TransferFailure
> 
> BUG=583343
> 
> Original issue's description:
> > Update device/usb and its Mojo interface for variable size ISO packets.
> >
> > To support the WebUSB API our underlying USB library needs to support
> > isochronous transfers with full control over the packet size. We also
> > need to know the completion status of each packet which was previously
> > not available.
> >
> > This patch updates the interface to match that provided by libusb and
> > the underlying platform specific APIs.
> >
> > BUG=492204
> >
> > Committed: https://crrev.com/051c98e9d3e843295d659b5676fcfa9dc1be5da6
> > Cr-Commit-Position: refs/heads/master@{#372844}
> 
> TBR=scheib@chromium.org,dgozman@chromium.org,reillyg@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=492204
> 
> Committed: https://crrev.com/c40c36ef3d3a722e0df920a4bbae25695f0b18e8
> Cr-Commit-Position: refs/heads/master@{#372971}

TBR=scheib@chromium.org,dgozman@chromium.org,zhaoqin@chromium.org
BUG=583343
==========

to

==========
Reland of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #1 id:1 of https://codereview.chromium.org/1658953003/ )

Reason for revert:
Fixed the uninitialized test buffer.

Original issue's description:
> Revert of Update device/usb and its Mojo interface for variable size ISO
packets. (patchset #3 id:120001 of https://codereview.chromium.org/1618393004/ )
>
> Reason for revert:
> Uninit error from UsbApiTest.TransferFailure
>
> BUG=583343
>
> Original issue's description:
> > Update device/usb and its Mojo interface for variable size ISO packets.
> >
> > To support the WebUSB API our underlying USB library needs to support
> > isochronous transfers with full control over the packet size. We also
> > need to know the completion status of each packet which was previously
> > not available.
> >
> > This patch updates the interface to match that provided by libusb and
> > the underlying platform specific APIs.
> >
> > BUG=492204
> >
> > Committed: https://crrev.com/051c98e9d3e843295d659b5676fcfa9dc1be5da6
> > Cr-Commit-Position: refs/heads/master@{#372844}
>
> TBR=scheib@chromium.org,dgozman@chromium.org,reillyg@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=492204
>
> Committed: https://crrev.com/c40c36ef3d3a722e0df920a4bbae25695f0b18e8
> Cr-Commit-Position: refs/heads/master@{#372971}

TBR=scheib@chromium.org,dgozman@chromium.org,zhaoqin@chromium.org
BUG=583343

Committed: https://crrev.com/ccbaed85d9538dc645535efdb58c0af6ebafdd43
Cr-Commit-Position: refs/heads/master@{#373065}
==========

[commit-bot: I haz the power, 2016-02-02 22:51:55]

Patchset 1 (id:??) landed as
https://crrev.com/ccbaed85d9538dc645535efdb58c0af6ebafdd43
Cr-Commit-Position: refs/heads/master@{#373065}