COOP+COEP data URL tests timing out
Categories
(Core :: DOM: Workers, defect, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox78 | --- | fixed |
People
(Reporter: annevk, Assigned: valentin)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
I haven't landed these yet (and maybe I should make some more readability tweaks) but https://github.com/web-platform-tests/wpt/pull/21146 has tests that are timing out in Fx Nightly. The blob ones all work, but the data ones do not (both dedicated worker and frame).
|
||
Updated•5 years ago
|
|
||
Updated•5 years ago
|
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 2•4 years ago
|
||
Hi Anne, it's not clear to me why the data URL tests don't work, but it looks like they time out in all browsers according to wpt.fyi
Am I reading this correctly?
|
Reporter | |
Comment 3•4 years ago
|
||
Now Chrome implements COEP it revealed a bug in the test that I addressed in https://github.com/web-platform-tests/wpt/pull/22928. With that fix Chrome once again finds all 18 tests (and fails a couple trivial ones), but Firefox continues to time out.
|
||
Updated•4 years ago
|
|
|
Assignee | |
Comment 4•4 years ago
|
||
The serialization of data URLs in firefox strips newlines from the string,
so the comments end up being on the same line with actual code.
This patch turns them into /* C style comments */
It also seems that the harness sometimes times out, even if all tests pass.
Pushed by valentin.gosu@gmail.com: https://hg.mozilla.org/integration/autoland/rev/c0c878881c86 Make data: tests in blob-data.https.html run r=annevk
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/23535 for changes under testing/web-platform/tests
Can't merge web-platform-tests PR due to failing upstream checks: Github PR https://github.com/web-platform-tests/wpt/pull/23535 * Community-TC (pull_request) (https://community-tc.services.mozilla.com/tasks/groups/Dt8Vuyc8QHuC45NMCtCPlA)
|
||
Comment 8•4 years ago
|
||
| bugherder | ||
|
|
Assignee | |
Updated•4 years ago
|
Upstream PR merged by jgraham
Description
•