Closed
Bug 649566
Opened 14 years ago
Closed 14 years ago
@keyframe(s) missing from <style> sanitizing code in nsHTMLFragmentContentSink
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla6
People
(Reporter: craig.topper, Assigned: ehsan.akhgari)
Details
Attachments
(1 file)
(deleted),
patch
|
bzbarsky
:
review+
dbaron
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
This code contains case statements for every other CSS rule type so should probably have the keyframe(s) rule types too.
Reporter | ||
Comment 1•14 years ago
|
||
The code is missing DOCUMENT_RULE too.
Comment 2•14 years ago
|
||
It should really also:
(a) have a default: case, and set didSanitize to true in that case
(b) handle a bunch of the rule classes it explicitly doesn't handle
Comment 3•14 years ago
|
||
Assignee | ||
Comment 4•14 years ago
|
||
Comment 5•14 years ago
|
||
Comment on attachment 526076 [details] [diff] [review]
Patch (v1)
r=me, but file a followup on maybe leaving the group rules in but sanitizing their contents?
Attachment #526076 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 6•14 years ago
|
||
Filed bug 650094.
CCing Henri as this patch probably steps on his tows...
(In reply to comment #7)
> CCing Henri as this patch probably steps on his tows...
Thanks. Whichever of the patch here and or the patch for bug 482909 lands first causes the patch for the other one to rot in such a way that eventually the fix here has to be rewritten on top of bug 482909.
Assignee | ||
Comment 8•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
Assignee | ||
Updated•14 years ago
|
status1.9.1:
--- → ?
status1.9.2:
--- → ?
status-firefox5:
--- → affected
tracking-firefox5:
--- → ?
Comment 9•14 years ago
|
||
Comment on attachment 526076 [details] [diff] [review]
Patch (v1)
a=dbaron for mozilla-aurora (from triage meeting)
Attachment #526076 -
Flags: approval-mozilla-aurora+
Assignee | ||
Comment 11•14 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•