Open
Bug 1511193
Opened 6 years ago
Updated 2 years ago
Green up and reenable skipped wpt content-security-policy/navigate-to/ tests on Android.
Categories
(Core :: DOM: Security, enhancement, P3)
Core
DOM: Security
Tracking
()
NEW
People
(Reporter: KWierso, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [domsecurity-backlog3])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
We recently enabled WPT on Fennec, and as part of that, had to skip some frequent intermittently failing tests to get a mostly-green run. Filing this bug to track hopefully fixing whatever flakiness is causing the intermittent failures so these tests can be reenabled.
Running the tests locally on repeat, these ones failed intermittently:
content-security-policy/navigate-to/child-navigates-parent-blocked.sub.html : TIMEOUT 4 out of 5 attempts
content-security-policy/navigate-to/unsafe-allow-redirects/allowed-end-of-chain-because-of-same-origin.sub.html : TIMEOUT 3 out of 5 attempts
content-security-policy/navigate-to/unsafe-allow-redirects/allowed-end-of-chain.sub.html : TIMEOUT 1 out of 5 attempts
content-security-policy/navigate-to/unsafe-allow-redirects/blocked-end-of-chain.sub.html : TIMEOUT 1 out of 5 attempts
Reporter | ||
Comment 1•6 years ago
|
||
Reporter | ||
Updated•6 years ago
|
Keywords: leave-open
Pushed by wkocher@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d9dd441d53fd
Update test annotations for fennec wpt csp/navigate-to/ a=testonly
Comment 3•6 years ago
|
||
bugherder |
Reporter | ||
Updated•6 years ago
|
Keywords: leave-open
Comment 4•6 years ago
|
||
Wes: This had leave-open, then something landed, and now it's not 'leave-open'. Does that mean it can be resolved? Or is there something more here?
Firefox does not currently support CSP navigate-to -- these tests should be skipped or annotated as expected fail.
Flags: needinfo?(wkocher)
Reporter | ||
Comment 5•6 years ago
|
||
This is not resolved, imo, though I don't have any strong feelings about keeping this bug open if that's problematic for triage in this component.
I had to skip these tests (rather than just annotate as failing) on android because they were frequently randomly failing in ways other than the expected (crash, timeout, notrun, etc), annotated failures expected.
Having the tests disabled (rather than annotated as expected failures) means we won't see if something starts making the tests pass (expected because we start supporting the feature or unexpected because something randomly tweaks things in ways that make the tests start passing).
In this case, though, we would probably see those changes based on the desktop tests, so if you want to resolve the bug, I won't stop you.
Flags: needinfo?(wkocher)
Comment 6•6 years ago
|
||
In that case we just move it into the backlog for now - thanks!
Priority: -- → P3
Whiteboard: [domsecurity-backlog3]
Summary: Green up and reenable skipped wpt content-security-policy/navigate-to/ tests on fennec. → Green up and reenable skipped wpt content-security-policy/navigate-to/ tests on Android.
Blocks: 1560374
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•