Move SVG shape painting code to SVGShapePainter

Source/core/rendering/svg/RenderSVGShape.h

Index: Source/core/rendering/svg/RenderSVGShape.h
diff --git a/Source/core/rendering/svg/RenderSVGShape.h b/Source/core/rendering/svg/RenderSVGShape.h
index 29876f714649e8a656b3ad50db349cfe5e2559a9..ac634d69d2a9352ecd2215e39f35cd3e44eefd46 100644
--- a/Source/core/rendering/svg/RenderSVGShape.h
+++ b/Source/core/rendering/svg/RenderSVGShape.h
@@ -36,7 +36,6 @@
 namespace blink {
 
 class FloatPoint;
-class GraphicsContextStateSaver;
 class PointerEventsHitRules;
 class SVGGraphicsElement;
 
@@ -48,8 +47,6 @@ public:
     void setNeedsShapeUpdate() { m_needsShapeUpdate = true; }
     virtual void setNeedsBoundariesUpdate() override final { m_needsBoundariesUpdate = true; }
     virtual void setNeedsTransformUpdate() override final { m_needsTransformUpdate = true; }
-    virtual void fillShape(GraphicsContext*) const;
-    virtual void strokeShape(GraphicsContext*) const;
 
     bool nodeAtFloatPointInternal(const HitTestRequest&, const FloatPoint&, PointerEventsHitRules);
 
@@ -58,20 +55,35 @@ public:
         ASSERT(m_path);
         return *m_path;
     }
+    bool hasPath() const { return m_path.get(); }
 
     virtual bool isShapeEmpty() const { return path().isEmpty(); }
 
+    virtual FloatRect paintInvalidationRectInLocalCoordinates() const override final { return m_paintInvalidationBoundingBox; }

[chrishtr, 2014/10/27 03:34:11]

Can this one be un-virtualized?

[chrishtr, 2014/10/27 03:34:12]

Can this one be un-virtualized?

[fs, 2014/10/27 12:36:27]

Not quite yet I think (still a few places calling this through RenderObject.)
Given the 'final' it ought to be devirtualized in the context of paint/ though.

+
+    bool hasNonScalingStroke() const { return style()->svgStyle().vectorEffect() == VE_NON_SCALING_STROKE; }
+    Path* nonScalingStrokePath(const Path*, const AffineTransform&) const;
+    AffineTransform nonScalingStrokeTransform() const;
+    virtual AffineTransform localTransform() const override final { return m_localTransform; }
+
+    virtual const Vector<MarkerPosition>* markerPositions() const { return 0; }
+
+    float strokeWidth() const;
+
+    virtual bool useRectRenderingFastPath() const { return false; }
+    virtual bool useEllipseRenderingFastPath() const { return false; }

[fs, 2014/10/27 12:36:27]

Why two extension points for this? They'll be mutually exclusive after all, so
something like:

enum GeometryCodePath {
  ...
};

virtual GeometryCodePath geometryCodePath() const { return ...; }

might do as well. (To go entirely crazy, that could just be stored in the
bitfield.)

[pdr., 2014/10/27 22:29:43]

Good idea, done.

+    virtual const Vector<FloatPoint>* zeroLengthLineCaps() const { return 0; };

[fs, 2014/10/27 12:36:27]

Nit: Redundant ';'.

[pdr., 2014/10/27 22:29:43]

Done.

+
+    virtual FloatRect objectBoundingBox() const override final { return m_fillBoundingBox; }
+
 protected:
     virtual void updateShapeFromElement();
     virtual bool shapeDependentStrokeContains(const FloatPoint&);
     virtual bool shapeDependentFillContains(const FloatPoint&, const WindRule) const;
-    float strokeWidth() const;
-    bool hasPath() const { return m_path.get(); }
     bool hasSmoothStroke() const;
 
-    bool hasNonScalingStroke() const { return style()->svgStyle().vectorEffect() == VE_NON_SCALING_STROKE; }
-    AffineTransform nonScalingStrokeTransform() const;
-    Path* nonScalingStrokePath(const Path*, const AffineTransform&) const;
+    // Give RenderSVGPath a hook for updating markers in updateShapeFromElement.
+    virtual void processMarkerPositions() { };

[fs, 2014/10/27 12:36:27]

Nit: Redundant ';'

 
     FloatRect m_fillBoundingBox;
     FloatRect m_strokeBoundingBox;
@@ -81,9 +93,7 @@ private:
     bool fillContains(const FloatPoint&, bool requiresFill = true, const WindRule fillRule = RULE_NONZERO);
     bool strokeContains(const FloatPoint&, bool requiresStroke = true);
 
-    virtual FloatRect paintInvalidationRectInLocalCoordinates() const override final { return m_paintInvalidationBoundingBox; }
     virtual const AffineTransform& localToParentTransform() const override final { return m_localTransform; }
-    virtual AffineTransform localTransform() const override final { return m_localTransform; }
 
     virtual bool isOfType(RenderObjectType type) const override { return type == RenderObjectSVGShape || RenderSVGModelObject::isOfType(type); }
     virtual const char* renderName() const override { return "RenderSVGShape"; }
@@ -94,25 +104,15 @@ private:
 
     virtual bool nodeAtFloatPoint(const HitTestRequest&, HitTestResult&, const FloatPoint& pointInParent, HitTestAction) override final;
 
-    virtual FloatRect objectBoundingBox() const override final { return m_fillBoundingBox; }
     virtual FloatRect strokeBoundingBox() const override final { return m_strokeBoundingBox; }
     FloatRect calculateObjectBoundingBox() const;
     FloatRect calculateStrokeBoundingBox() const;
     void updatePaintInvalidationBoundingBox();
 
-    bool setupNonScalingStrokeContext(AffineTransform&, GraphicsContextStateSaver&);
-
-    bool shouldGenerateMarkerPositions() const;
-    FloatRect markerRect(float strokeWidth) const;
-    void processMarkerPositions();
-
-    void paintMarkers(PaintInfo&);
-
 private:
     FloatRect m_paintInvalidationBoundingBox;
     AffineTransform m_localTransform;
     OwnPtr<Path> m_path;
-    Vector<MarkerPosition> m_markerPositions;
 
     bool m_needsBoundariesUpdate : 1;
     bool m_needsShapeUpdate : 1;