Bug 91131
| Summary: | Move markers from RenderSVGShape to RenderSVGPath | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Philip Rogers <pdr> |
| Component: | SVG | Assignee: | Philip Rogers <pdr> |
| Status: | ASSIGNED | ||
| Severity: | Normal | CC: | fmalita, krit, schenney, zimmermann |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 88231 | ||
Philip Rogers
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
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
(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?