1397655 - Update tests within dom/ to comply with new toplevel data: URI navigation policy

Status: RESOLVED FIXED

(Core :: DOM: Security, enhancement, P2)

Comment 1

[Christoph Kerschbaumer [:ckerschb]]

Attachment: bug_1397655_dom_tests.patch

Smaug, overall we are facing several failing tests within docshell/ when flipping the pref |security.data_uri.block_toplevel_data_uri_navigations|:
* dom/base/test/browser_bug1303838.js
* dom/base/test/test_data_uri.html
* dom/base/test/test_x-frame-options.html
* dom/file/tests/test_ipc_messagemanager_blob.html
* dom/html/test/test_fullscreen-api-race.html
* dom/ipc/tests/test_cpow_cookies.html
* dom/plugins/test/mochitest/test_pluginstream_err.html

Some explanations:
I think dom/base/test/test_data_uri.html should be testing the data URI, hence I think it makes sense to explicitly set the pref to false.

For dom/base/test/test_x-frame-options.html, the is e.g. this comment '// test that a document can be framed under a data: URL opened by the same site as the frame', hence I think it makes more sense to explicitly push the pref for the test, if you feel different, please let me know.

For the test test/test_fullscreen-api-race.html, we already set the pref |security.data_uri.unique_opaque_origin|, hence I think we should also set the pref explicitly for this test.

To be honest, I don't know how to easily rewrite dom/plugins/test/mochitest/test_pluginstream_err.html without putting in too much effort, would you be fine with accepting the updated test where I just push the pref explicitly?

Seems there are no more dom/ tests that need to be rewritten:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=cb517e30d8209331f929f9a3f9fd2b61a5fed63c

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/697d93c85b77
Update tests within dom/ to comply with new toplevel data: URI navigation policy. r=smaug

Comment 3

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/697d93c85b77