Closed Bug 1298749 Opened 8 years ago Closed 6 years ago

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

Categories

(Core :: DOM: CSS Object Model, defect)

Product:
Core
Component:
DOM: CSS Object Model
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: heycam, Unassigned)

Details

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.
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)
This wasn't urgent, and the code is now gone. :-)
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(cam)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.