246474 - deCOMtaminate SVG

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Jonathan Watt [:jwatt]]

Tracker bug. Please set any bugs relating to the deCOMtamination of the SVG code
to block this bug.

Comment 1

[Jonathan Watt [:jwatt]]

Mass reassign of SVG bugs that aren't currently being worked on by Alex to
[e-mail]. 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. 

Comment 2

[daniel newton]

Attachment: first stab at http://wiki.mozilla.org/SVG:Cleanup#DeCOMtaminate.2Fdevirtualize_cairo.2FThebes_renderer

Comment 3

[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 4

[tor]

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.