Closed Bug 1593835 Opened 5 years ago Closed 4 years ago

[wpt-sync] Sync PR 20078 - Rewrite WPT WebAudio tests to work on Chrome

Categories

(Core :: Web Audio, task, P4)

task

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Raymond Toy <rtoy@chromium.org> wrote:

Rewrite WPT WebAudio tests to work on Chrome

Chrome times out on these tests because the test throws an exception
in a promise resolver. Rewrite the tests to use the promise_test
method which handles these.

Manually verified that Firefox still passes these tests, and that
Chrome fails them instead of timing out. TestExpectations updated
appropriately.

Bug: 1011080, 1021295
Change-Id: Ib65dec0c49ad6c9764b4432c1e28282f8a7d268a
Reviewed-on: https://chromium-review.googlesource.com/1898606
WPT-Export-Revision: 2dd9cc2bf6bed61a62779cd887a6576b9f0be75c

Component: web-platform-tests → Web Audio
Product: Testing → Core

GitHub CI Results

wpt.fyi PR Results Base Results

Ran 4 tests and 4 subtests

Firefox

OK : 3
PASS : 2
FAIL : 1

Chrome

OK : 3
FAIL : 3

Safari

OK : 3
FAIL : 3

Existing tests that now have a worse result

/webaudio/the-audio-api/processing-model/cycle-witout-delay.html: Firefox: OK->MISSING, Chrome: ERROR->MISSING, Safari: OK->MISSING
Test that cycles that don't contain a DelayNode are muted: Firefox: PASS->MISSING, Chrome: TIMEOUT->MISSING, Safari: FAIL->MISSING

/webaudio/the-audio-api/processing-model/delay-time-clamping.html
Test that a DelayNode allows a feedback loop of a single rendering quantum: Firefox: PASS->FAIL, Chrome: TIMEOUT->FAIL, Safari: FAIL->FAIL

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a9d0fa260f2c
[wpt PR 20078] - Rewrite WPT WebAudio tests to work on Chrome, a=testonly
https://hg.mozilla.org/integration/autoland/rev/409fffa05355
[wpt PR 20078] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.