[webkit-reviews] review denied: [Bug 20699] SVGPaintServer CleanUp : [Attachment 23625] SVGPaintServer cleanUp - third part

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has denied Dirk Schulze <vbs85 at gmx.de>'s request
for review:
Bug 20699: SVGPaintServer CleanUp
https://bugs.webkit.org/show_bug.cgi?id=20699

Attachment 23625: SVGPaintServer cleanUp - third part
https://bugs.webkit.org/attachment.cgi?id=23625&action=edit

------- Additional Comments from Eric Seidel <eric at webkit.org>
Looks fine.  Except why are you adding:
 97 void SVGPaintServerSolid::renderPath(GraphicsContext*& context, const
RenderObject* path, SVGPaintTargetType type) const

That's already implemented exactly like that in SVGPaintServer, or?  Maybe
that's only for CG?

Also, it seems:
context->setFillRule(svgStyle->fillRule());

should go in setup() in the fill if-block not in renderPath(), or?