Closed Bug 1205654 Opened 4 years ago Closed 4 years ago

wpt mixed-content + picture tag tests unstable with e10s

Categories

(Core :: DOM: Security, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
e10s + ---
firefox48 --- fixed

People

(Reporter: jgraham, Assigned: RyanVM)

References

(Blocks 1 open bug)

Details

(Whiteboard: [domsecurity-intermittent])

Attachments

(1 file)

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
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
(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)
Whiteboard: [domsecurity-active]
Whiteboard: [domsecurity-active] → [domsecurity-intermittent]
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
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)
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: mozilla → ryanvm
Blocks: 1222277
https://hg.mozilla.org/mozilla-central/rev/9c042b16ca2c
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.