Closed
Bug 1205654
Opened 9 years ago
Closed 8 years ago
wpt mixed-content + picture tag tests unstable with e10s
Categories
(Core :: DOM: Security, defect)
Core
DOM: Security
Tracking
()
RESOLVED
FIXED
mozilla48
People
(Reporter: jgraham, Assigned: RyanVM)
References
(Blocks 1 open bug)
Details
(Whiteboard: [domsecurity-intermittent])
Attachments
(1 file)
17.70 KB,
patch
|
jgraham
:
review+
|
Details | Diff | Splinter Review |
testing/web-platform/meta/mixed-content/allowed/http-csp/same-host-https/picture-tag/top-level/keep-scheme-redirect/allowed.https.html.ini testing/web-platform/meta/mixed-content/allowed/http-csp/same-host-https/picture-tag/top-level/no-redirect/allowed.https.html.ini testing/web-platform/meta/mixed-content/allowed/meta-csp/same-host-https/picture-tag/top-level/no-redirect/allowed.https.html.ini testing/web-platform/meta/mixed-content/allowed/no-opt-in/same-host-https/picture-tag/top-level/keep-scheme-redirect/allowed.https.html.ini testing/web-platform/meta/mixed-content/allowed/no-opt-in/same-host-https/picture-tag/top-level/no-redirect/allowed.https.html.ini testing/web-platform/meta/mixed-content/blockable/http-csp/cross-origin-http/picture-tag/top-level/no-redirect/opt-in-blocks.https.html.ini testing/web-platform/meta/mixed-content/blockable/http-csp/same-host-http/picture-tag/top-level/no-redirect/opt-in-blocks.https.html.ini testing/web-platform/meta/mixed-content/blockable/no-opt-in/same-host-http/picture-tag/top-level/no-redirect/no-opt-in-blocks.https.html.ini Disabled for now
Updated•9 years ago
|
tracking-e10s:
--- → +
Assignee | ||
Comment 1•8 years ago
|
||
Christoph, there's a lot of disabled CSP web-platform-tests on e10s from the looks of it. Can you please investigate?
Component: web-platform-tests → DOM: Security
Flags: needinfo?(mozilla)
Product: Testing → Core
Comment 2•8 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #1) > Christoph, there's a lot of disabled CSP web-platform-tests on e10s from the > looks of it. Can you please investigate? Thanks Ryan - I'll have a look.
Assignee: nobody → mozilla
Status: NEW → ASSIGNED
Flags: needinfo?(mozilla)
Updated•8 years ago
|
Whiteboard: [domsecurity-active]
Updated•8 years ago
|
Whiteboard: [domsecurity-active] → [domsecurity-intermittent]
Comment 3•8 years ago
|
||
Let's see if we can reproduce that on TRY [1] so we can browse through the logs to see what's going on. [1] https://treeherder.mozilla.org/#/jobs?repo=try&revision=1f4b9fc766e4
Assignee | ||
Comment 4•8 years ago
|
||
This patch cleans up these tests and also takes care of bug 1222277 too. At this point, I'd like to close this bug out and leave the remaining issues with the mixed-content/blockable tests to bug 1184351.
Attachment #8736986 -
Flags: review?(james)
Assignee | ||
Comment 5•8 years ago
|
||
Non-e10s: https://treeherder.mozilla.org/#/jobs?repo=try&revision=06a259bf56b5 e10s: https://treeherder.mozilla.org/#/jobs?repo=try&revision=80901626439d (I have earlier Try pushes with all-green on Windows as well)
Reporter | ||
Comment 6•8 years ago
|
||
Comment on attachment 8736986 [details] [diff] [review] re-enable most of the tests on e10s Review of attachment 8736986 [details] [diff] [review]: ----------------------------------------------------------------- Thanks!
Attachment #8736986 -
Flags: review?(james) → review+
Assignee | ||
Updated•8 years ago
|
Assignee: mozilla → ryanvm
Comment 8•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9c042b16ca2c
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•