Closed
Bug 246474
Opened 20 years ago
Closed 15 years ago
deCOMtaminate SVG
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
People
(Reporter: jwatt, Unassigned)
Details
(Keywords: meta)
Attachments
(1 file)
Tracker bug. Please set any bugs relating to the deCOMtamination of the SVG code to block this bug.
Reporter | ||
Comment 1•20 years ago
|
||
Mass reassign of SVG bugs that aren't currently being worked on by Alex to general@svg.bugs. If you think someone should be assigned to your bug you can join the #svg channel on mozilla.org's IRC server ( irc://irc.mozilla.org/svg ) where you can try to convince one of the SVG hackers to work on it. We aren't always there, so if you don't get a response straight away please try again later.
Assignee: alex → general
Comment 2•18 years ago
|
||
Updated•18 years ago
|
Attachment #239463 -
Flags: review?(tor)
Thanks for taking a look at this problem. While this is certainly one approach down the path to eventually switching to Thebes, we've been following an alternate strategy of removing the renderer classes. So far we've reduced it from the eight still used in the 1.8 branch to just two - the canvas and surface. nsISVGRendererSurface removal is well underway, with it already gone from both <image> and <filter> (bug 341738 and bug 353827 respectively) and removal from svg effects (bug 331130) and patterns (bug 354163) coming soon. At that point we'll be left with just nsISVGRendererCanvas, which we could deCOM with your approach if Thebes isn't ready for SVG yet. The main decider for the latter is progress of the gfxTextRun work needed to support textPath.
Comment on attachment 239463 [details] [diff] [review] first stab at http://wiki.mozilla.org/SVG:Cleanup#DeCOMtaminate.2Fdevirtualize_cairo.2FThebes_renderer Remove old review request. I think similar changes for most of this patch have already landed.
Attachment #239463 -
Flags: review?(tor)
Reporter | ||
Updated•15 years ago
|
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•