Closed Bug 1552441 Opened 3 years ago Closed 2 years ago

Write test to verify only non linkable about: pages can load local module scripts

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: ckerschb, Assigned: jaws)

References

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

No description provided.

Jared, I am OOO all of next week. Any chance you could provide the test here? :smaug requested one and to be fair, I think it really makes sense to have one. We could then land this bug together with the actual fix within Bug 1551379.

Any chance you could provide that test?

Type: defect → task
Flags: needinfo?(jaws)
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Flags: needinfo?(jaws)

Note, this test does not pass currently with the patch from bug 1551379. Both the unlinkable and linkable pages load the script and the page's content gets overwritten.

Priority: -- → P2
Whiteboard: [domsecurity-active]
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/158eb97dfd5e
Test to verify that only unlinkable about pages can load script[type=module]. r=smaug,ckerschb

Backed out changeset 158eb97dfd5e (bug 1552441) for browser-chrome failures in dom/tests/browser/browser_unlinkable_about_page_can_load_module_scripts.js. CLOSED TREE

Log:
https://treeherder.mozilla.org/logviewer.html#?job_id=247405406&repo=autoland

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=158eb97dfd5e6b18f6a13cbf4e648b361deb1a41

Backout:
https://hg.mozilla.org/integration/autoland/rev/eb1763edca85fb806e76a65a7a7a9ea80bf19a65

Flags: needinfo?(jaws)
Flags: needinfo?(jaws)
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/60bf0ae446ce
Test to verify that only unlinkable about pages can load script[type=module]. r=smaug,ckerschb
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Flags: qe-verify-
Regressions: 1559087
You need to log in before you can comment on or make changes to this bug.