Perma webaudio/the-audio-api/the-audioworklet-interface/audioworkletnode-construction.https.html | X Creating a node before loading a module should throw. threw "ReferenceError" instead of InvalidStateError. when Gecko 71 merges to Beta on 2019-10-14
Categories
(Core :: Web Audio, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox-esr68 | --- | unaffected |
firefox69 | --- | unaffected |
firefox70 | --- | unaffected |
firefox71 | + | verified |
People
(Reporter: rmaries, Assigned: aryx)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
[Tracking Requested - why for this release]:
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=269586363&repo=try&lineNumber=20329
[task 2019-10-03T11:19:49.728Z] 11:19:49 INFO - TEST-OK | /webaudio/the-audio-api/the-audioworklet-interface/audioworkletnode-channel-count.https.html | took 1353ms
[task 2019-10-03T11:19:49.728Z] 11:19:49 INFO - TEST-START | /webaudio/the-audio-api/the-audioworklet-interface/audioworkletnode-construction.https.html
[task 2019-10-03T11:19:49.732Z] 11:19:49 INFO - Closing window 43
[task 2019-10-03T11:19:49.773Z] 11:19:49 INFO - PID 26049 | -----------------------------------------------------
[task 2019-10-03T11:19:49.782Z] 11:19:49 INFO - PID 26049 | Suppressions used:
[task 2019-10-03T11:19:49.782Z] 11:19:49 INFO - PID 26049 | count bytes template
[task 2019-10-03T11:19:49.783Z] 11:19:49 INFO - PID 26049 | 26 800 nsComponentManagerImpl
[task 2019-10-03T11:19:49.784Z] 11:19:49 INFO - PID 26049 | 611 17713 libfontconfig.so
[task 2019-10-03T11:19:49.785Z] 11:19:49 INFO - PID 26049 | 1 29 libglib-2.0.so
[task 2019-10-03T11:19:49.785Z] 11:19:49 INFO - PID 26049 | -----------------------------------------------------
[task 2019-10-03T11:19:50.888Z] 11:19:50 INFO -
[task 2019-10-03T11:19:50.888Z] 11:19:50 INFO - TEST-PASS | /webaudio/the-audio-api/the-audioworklet-interface/audioworkletnode-construction.https.html | # AUDIT TASK RUNNER STARTED.
[task 2019-10-03T11:19:50.888Z] 11:19:50 INFO - TEST-PASS | /webaudio/the-audio-api/the-audioworklet-interface/audioworkletnode-construction.https.html | > [construction-before-module-loading]
[task 2019-10-03T11:19:50.888Z] 11:19:50 INFO - TEST-UNEXPECTED-FAIL | /webaudio/the-audio-api/the-audioworklet-interface/audioworkletnode-construction.https.html | X Creating a node before loading a module should throw. threw "ReferenceError" instead of InvalidStateError. - assert_true: expected true got false
[task 2019-10-03T11:19:50.888Z] 11:19:50 INFO - _logFailed/<@https://web-platform.test:8443/webaudio/resources/audit.js:50:18
[task 2019-10-03T11:19:50.888Z] 11:19:50 INFO - Test.prototype.step@https://web-platform.test:8443/resources/testharness.js:1908:25
[task 2019-10-03T11:19:50.888Z] 11:19:50 INFO - test@https://web-platform.test:8443/resources/testharness.js:544:30
[task 2019-10-03T11:19:50.889Z] 11:19:50 INFO - _logFailed@https://web-platform.test:8443/webaudio/resources/audit.js:49:9
[task 2019-10-03T11:19:50.889Z] 11:19:50 INFO - _finalize@https://web-platform.test:8443/webaudio/resources/audit.js:215:19
[task 2019-10-03T11:19:50.889Z] 11:19:50 INFO - _assert@https://web-platform.test:8443/webaudio/resources/audit.js:228:12
[task 2019-10-03T11:19:50.889Z] 11:19:50 INFO - throw@https://web-platform.test:8443/webaudio/resources/audit.js:337:19
[task 2019-10-03T11:19:50.889Z] 11:19:50 INFO - @https://web-platform.test:8443/webaudio/the-audio-api/the-audioworklet-interface/audioworkletnode-construction.https.html:26:18
[task 2019-10-03T11:19:50.889Z] 11:19:50 INFO - run@https://web-platform.test:8443/webaudio/resources/audit.js:1195:14
[task 2019-10-03T11:19:50.889Z] 11:19:50 INFO - _runNextTask@https://web-platform.test:8443/webaudio/resources/audit.js:1275:67
Reporter | ||
Comment 1•5 years ago
|
||
James, I think bug 1584570, https://hg.mozilla.org/mozilla-central/rev/99895cb320b0 could be the culprit. Coud you take a look?
Comment 2•5 years ago
|
||
These tests are breaking the supposed invariant that you return the same subtests on pass and fail. So on beta we get test names that simply don't exist on nightly, and so the metadata update optimises out the corresponding metadata.
I think there's going to have to be a solution for this kind of thing in the metadata, but really it would be good to fix the test to use the harness in vaugely the correct way. ni karlt for that.
In the interim feel free to revert the corresponding metadata changes on beta.
Comment 3•5 years ago
|
||
https://phabricator.services.mozilla.com/D47025 is one part of changes required to make test names consistent.
Happy to fix other parts if that approach is accepted.
https://github.com/web-platform-tests/wpt/issues/10201
Updated•5 years ago
|
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Updated•5 years ago
|
Comment 6•5 years ago
|
||
(In reply to Karl Tomlinson (back Oct 14 :karlt) from comment #3)
https://phabricator.services.mozilla.com/D47025 is one part of changes required to make test names consistent.
Happy to fix other parts if that approach is accepted.
https://github.com/web-platform-tests/wpt/issues/10201
Hi Karl, I saw the above landed (phab patch), is there an update here?
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 7•5 years ago
|
||
The annotations got removed in bug 1584570 (due an issue with the tool which
updates the annotations). All of these annotations had been added before, most
in bug 1583496.
The test fail because AudioWorklet is not enabled on these trees.
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 9•5 years ago
|
||
Comment 10•5 years ago
|
||
Comment 11•5 years ago
|
||
bugherder |
Comment hidden (Intermittent Failures Robot) |
Comment 13•5 years ago
|
||
No occurrences since the patch landed: https://treeherder.mozilla.org/#/jobs?repo=try&selectedJob=271110034&resultStatus=usercancel%2Ctestfailed%2Cbusted%2Cexception%2Crunnable&revision=3641cb99f0591a748ccc93769012a0f4ddc9c2c3&searchStr=wpt
Comment 14•5 years ago
|
||
(In reply to Andreea Pavel [:apavel] from comment #6)
(In reply to Karl Tomlinson (back Oct 14 :karlt) from comment #3)
https://phabricator.services.mozilla.com/D47025 is one part of changes required to make test names consistent.
Hi Karl, I saw the above landed (phab patch), is there an update here?
It has not landed. It is still waiting for review.
Comment 15•5 years ago
|
||
(In reply to Karl Tomlinson (:karlt) from comment #14)
(In reply to Andreea Pavel [:apavel] from comment #6)
(In reply to Karl Tomlinson (back Oct 14 :karlt) from comment #3)
https://phabricator.services.mozilla.com/D47025 is one part of changes required to make test names consistent.
Hi Karl, I saw the above landed (phab patch), is there an update here?
It has not landed. It is still waiting for review.
Sorry, this is misleading https://lando.services.mozilla.com/D47025/
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•5 years ago
|
Comment 19•5 years ago
|
||
bugherder uplift |
Reporter | ||
Updated•5 years ago
|
Updated•3 years ago
|
Description
•