Closed
Bug 1652351
Opened 4 years ago
Closed 4 years ago
IsHTMLDocument check in FeaturePolicy looks wrong.
Categories
(Core :: DOM: Security, defect, P2)
Core
DOM: Security
Tracking
()
RESOLVED
FIXED
mozilla80
| Tracking | Status | |
|---|---|---|
| firefox80 | --- | fixed |
People
(Reporter: emilio, Assigned: ckerschb)
References
(Regression)
Details
(Keywords: regression, Whiteboard: [domsecurity-active])
Attachments
(1 file)
Why should an XML or XHTML or SVG document have access to all the features?
Andrea, is there something I'm missing? Or should we just get rid of that check?
Flags: needinfo?(amarchesini)
|
Reporter | |
Comment 1•4 years ago
|
||
|
||
Comment 2•4 years ago
|
||
Good catch. That check seems wrong to me. I don't recall why I had added it. I'm OK with removing it.
Flags: needinfo?(amarchesini)
|
Assignee | |
Comment 3•4 years ago
|
||
|
||
Updated•4 years ago
|
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•4 years ago
|
|
||
Updated•4 years ago
|
Has Regression Range: --- → yes
Pushed by btara@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ac47fa4e5e44
Remove incorrect IsHTMLDocument check within FeaturePolicyUtils::IsFeatureAllowed for FeaturePolicy. r=baku
|
||
Comment 5•4 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox80:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
|
||
Updated•3 years ago
|
Keywords: regression
You need to log in
before you can comment on or make changes to this bug.
Description
•