Closed Bug 1652351 Opened 4 years ago Closed 4 years ago

IsHTMLDocument check in FeaturePolicy looks wrong.

Categories

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

defect

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)

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: nobody → ckerschb
Status: NEW → ASSIGNED
Severity: -- → S2
Priority: -- → P2
Regressed by: 1483631
Whiteboard: [domsecurity-active]
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
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: