ASSIGNED91131
Move markers from RenderSVGShape to RenderSVGPath 
https://bugs.webkit.org/show_bug.cgi?id=91131
Summary Move markers from RenderSVGShape to RenderSVGPath
Philip Rogers
Reported 2012-07-12 11:56:40 PDT
The SVG spec only allows markers on paths but our current codebase has the code in RenderSVGShape, the superclass of RenderSVGRect and RenderSVGEllipse. Neither rects nor ellipses can have markers so we should move the marker code into RenderSVGPath for better code clarity and rendering performance.
Attachments
Add attachment proposed patch, testcase, etc.
Stephen Chenney
Comment 1 2012-07-12 12:05:38 PDT
The SVG 2.0 spec will allow more variants of markers, and I think it includes markers an any shape. Check before doing this.
Philip Rogers
Comment 2 2012-07-12 16:59:03 PDT
(In reply to comment #1) > The SVG 2.0 spec will allow more variants of markers, and I think it includes markers an any shape. Check before doing this. I poked around the W3C and I think what is being proposed is more general markers (essentially, <use>) and stuff like shapes along paths but not markers along shapes. It does make sense to use shapes to stroke a shape though, so I will hold off on this patch for now. Dirk, Niko, do either of you know more information?
Note You need to log in before you can comment on or make changes to this bug.