Skip to content

Commit

Permalink
Bug 1924455 [wpt PR 48591] - [wpt] Fail `VideoTrackGenerator-with-win…
Browse files Browse the repository at this point in the history
…dow-tracks.https.html` faster, a=testonly

Automatic update from web-platform-tests
[wpt] Fail `VideoTrackGenerator-with-window-tracks.https.html` faster

`VideoTrackGenerator` is not exposed to workers, which currently causes
the test to hang. Surface worker errors to the `promise_test()`s
instead.

Drive-by: always revoke the temporary script URL.

Bug: 40058895
Change-Id: Id0e3830abadecac7c76853235a09b201fdad1af0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5922435
Auto-Submit: Jonathan Lee <jonathanjleegoogle.com>
Reviewed-by: Harald Alvestrand <htachromium.org>
Commit-Queue: Jonathan Lee <jonathanjleegoogle.com>
Cr-Commit-Position: refs/heads/main{#1368294}

--

wpt-commits: 302e6e04c6c54bf3891e6a6a15a2e04e3f72bb52
wpt-pr: 48591

UltraBlame original commit: 378f7e6ed8f4dcc66625b9b97b785562ec614a6a
  • Loading branch information
marco-c committed Oct 22, 2024
1 parent 2d54c86 commit 45a02a7
Showing 1 changed file with 24 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -573,39 +573,58 @@
url
)
;
try
{
await
new
Promise
(
(
resolve
reject
)
=
>
{
worker
.
onmessage
=
resolve
;
worker
.
onerror
=
(
err
)
=
>
{
resolve
reject
(
err
.
message
)
;
}
)
;
return
worker
;
}
finally
{
URL
.
revokeObjectURL
(
url
)
;
return
worker
;
}
}
promise_test
(
Expand Down

0 comments on commit 45a02a7

Please sign in to comment.