1686473 - [wpt-sync] Sync PR 27140 - usb: Expand tests for USB transfer completion order

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 27140 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/27140
Details from upstream follow.

b'Reilly Grant <[e-mail]>' wrote:

usb: Expand tests for USB transfer completion order

This change expands the manual test for transferIn() to send much more
data and assert that many overlapping transfers will still complete in
the correct order.

Two changes are necessary to make these tests pass. First, when aborting
a transfer on device close the ERROR_REQUEST_ABORTED should be mapped to
an AbortError DOMException. Second, transfers should be tracked in a
list rather than a map so that iteration order is well defined.

Bug: 1153647
Change-Id: I248ce4724e391216783a6873809aca43e4690b96
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2591928
Commit-Queue: Reilly Grant \<[e-mail]>
Reviewed-by: James Hollyer \<[e-mail]>
Cr-Commit-Position: refs/heads/master@{#842380}

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

PR 27140 applied with additional changes from upstream: 28ac8ff82f255bfb5c799b6d433d19d0a0eb0e34

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

PR 27140 applied with additional changes from upstream: 28ac8ff82f255bfb5c799b6d433d19d0a0eb0e34

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=b273aa0e7bb63d0dd13bebfcaa9eef7b2b90c1e9

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/afb64a85aad3
[wpt PR 27140] - usb: Expand tests for USB transfer completion order, a=testonly

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 6

[James Graham [:jgraham]]

Attachment: Bug 1686473 [wpt PR 27140] - usb: Expand tests for USB transfer completion order, a=testonly

Automatic update from web-platform-tests
usb: Expand tests for USB transfer completion order

This change expands the manual test for transferIn() to send much more
data and assert that many overlapping transfers will still complete in
the correct order.

Two changes are necessary to make these tests pass. First, when aborting
a transfer on device close the ERROR_REQUEST_ABORTED should be mapped to
an AbortError DOMException. Second, transfers should be tracked in a
list rather than a map so that iteration order is well defined.

Bug: 1153647
Change-Id: I248ce4724e391216783a6873809aca43e4690b96
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2591928
Commit-Queue: Reilly Grant <[e-mail]>
Reviewed-by: James Hollyer <[e-mail]>
Cr-Commit-Position: refs/heads/master@{#842380}

--

wpt-commits: 80090f6edd23993e9907dbc9fb1c6af7aebf91e7
wpt-pr: 27140

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/396b8a1e4655
[wpt PR 27140] - usb: Expand tests for USB transfer completion order, a=testonly

Comment 9

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/396b8a1e4655