Update tests within devtools/ to comply with new toplevel data: URI navigation policy

RESOLVED FIXED in Firefox 57

Status

()

enhancement
P1
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: ckerschb, Assigned: ckerschb)

Tracking

unspecified
mozilla57
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 fixed)

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 attachment)

No description provided.
Blocks: 1380959
Priority: -- → P3
Whiteboard: [domsecurity-backlog1]
Brian, we are about to block toplevel data: URI navigations within Firefox (because those navigations are mostly only used for phishing attempts). Flipping the pref |security.data_uri.block_toplevel_data_uri_navigations| causes two tests within devtools/ to fail:
* browser_markup_update-on-navigtion.js
* test_saveHeapSnapshot_e10s_01.html

Please note that even though the updated code within test_saveHeapSnapshot_e10s_01.html looks like it's an iframe, it's created through mozbrowser, which basically creates a new toplevel context, hence it would be blocked by the pref.
Attachment #8905413 - Flags: review?(bgrinstead)
Priority: P3 → P1
Whiteboard: [domsecurity-backlog1] → [domsecurity-active]
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Attachment #8905413 - Flags: review?(bgrinstead) → review+

Comment 2

2 years ago
Pushed by mozilla@christophkerschbaumer.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/860d5124faec
Update tests within devtools/ to comply with new toplevel data: URI navigation policy. r=bgrinstead
https://hg.mozilla.org/mozilla-central/rev/860d5124faec
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.