Closed
Bug 1843002
Opened 2 years ago
Closed 2 years ago
Add nonce and parser-metadata to ScriptFetchOptions
Categories
(Core :: DOM: Core & HTML, task)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
117 Branch
| Tracking | Status | |
|---|---|---|
| firefox117 | --- | fixed |
People
(Reporter: tschuster, Assigned: tschuster)
References
Details
Attachments
(2 files)
In the specification Script fetch options contains the cryptographic nonce and the parser metadata. Our implementation of ScriptFetchOptions however doesn't and instead queries the context element. When preloading we don't have an element so we need to provide this information explicitly.
| Assignee | ||
Comment 1•2 years ago
|
||
| Assignee | ||
Comment 2•2 years ago
|
||
Depends on D183365
Pushed by tschuster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d381b6365111
Add nonce to ScriptFetchOptions. r=allstarschh,smaug
https://hg.mozilla.org/integration/autoland/rev/21a9bee8f772
Add parser-metadata to ScriptFetchOptions. r=allstarschh,smaug
Comment 4•2 years ago
|
||
Backed out for causing build bustages in ScriptLoadRequest.h.
- Backout link
- Push with failures
- Failure Log
- Failure line: /builds/worker/workspace/obj-build/dist/include/js/loader/ScriptLoadRequest.h:302:3: error: declaration of 'JS::loader::ParserMetadata JS::loader::ScriptLoadRequest::ParserMetadata() const' [-fpermissive]
Flags: needinfo?(tschuster)
Pushed by tschuster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ff52f02b1f93
Add nonce to ScriptFetchOptions. r=allstarschh,smaug
https://hg.mozilla.org/integration/autoland/rev/7fdd6b03d37d
Add parser-metadata to ScriptFetchOptions. r=allstarschh,smaug
Comment 6•2 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/ff52f02b1f93
https://hg.mozilla.org/mozilla-central/rev/7fdd6b03d37d
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox117:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
| Assignee | ||
Updated•2 years ago
|
Flags: needinfo?(tschuster)
You need to log in
before you can comment on or make changes to this bug.
Description
•