nsCSSKeyframesRule::AppendRule should pass in a base URI / principal to ParseKeyframeRule

RESOLVED INVALID

Status

()

RESOLVED INVALID
2 years ago
7 months ago

People

(Reporter: heycam, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

2 years ago
nsCSSParser::ParseKeyframeRule only has as single nsIURI* argment, but we currently pass in null for it.  We also don't pass in a principal.  This causes any URL-valued things in the parsed rule to be dropped.
(Reporter)

Comment 1

2 years ago
That current nsIURI* argument is the sheet URI (used for error reporting).  We should have a second base URI argument.
Is this urgent? Are you going to fix it? :)
Flags: needinfo?(cam)
(Reporter)

Comment 3

7 months ago
This wasn't urgent, and the code is now gone. :-)
Status: NEW → RESOLVED
Last Resolved: 7 months ago
Flags: needinfo?(cam)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.