frame-ancestors tests time out waiting for load event on CSP-blocked resources
Categories
(Core :: DOM: Security, defect, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox75 | --- | fixed |
People
(Reporter: jgraham, Assigned: jgraham)
References
(Depends on 1 open bug)
Details
(Whiteboard: [domsecurity-active], [wptsync upstream])
Attachments
(1 file)
This means that the tests end up testing nothing, and take 5 minutes per platform of machine time (not) doing it.
|
Assignee | |
Comment 1•4 years ago
|
||
These tests are all timing out in Firefox because we don't send either
a load or an error event for CSP-blocked loads. To work around this,
poll the iframe for the load, assuming it's complete when we see a
non-about:blank document with readyState complete (or an exception
from trying to access a cross-origin resource).
|
||
Updated•4 years ago
|
|
||
Updated•4 years ago
|
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/autoland/rev/a2bf92004940 Poll for load in CSP frame-ancestors tests, r=bzbarsky
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/21757 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
|
||
Comment 5•4 years ago
|
||
Backed out for failing frame-ancestors-from-serviceworker.https.html
Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=a2bf92004940994b68132da4d82c3965af991535&selectedJob=288564538
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=288564538&repo=autoland&lineNumber=4218
Backout: https://hg.mozilla.org/integration/autoland/rev/b6ffa8f92a693f01e0bdc9b6bab67b40036e29bd
Upstream PR was closed without merging
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/autoland/rev/25bfc69bd9cc Poll for load in CSP frame-ancestors tests, r=bzbarsky
|
|
Assignee | |
Updated•4 years ago
|
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
|
||
Comment 9•4 years ago
|
||
| bugherder | ||
Upstream PR merged by moz-wptsync-bot
Description
•