Move 'closePath' handling to the consumers

Source/core/svg/SVGPathBuilder.cpp

Index: Source/core/svg/SVGPathBuilder.cpp
diff --git a/Source/core/svg/SVGPathBuilder.cpp b/Source/core/svg/SVGPathBuilder.cpp
index f708c2f5196246f6e17e5b748e31487ef53fcb51..c11335852dd0144bff6f7bfd03e5f081685991e2 100644
--- a/Source/core/svg/SVGPathBuilder.cpp
+++ b/Source/core/svg/SVGPathBuilder.cpp
@@ -28,12 +28,13 @@
 
 namespace blink {
 
-void SVGPathBuilder::moveTo(const FloatPoint& targetPoint, bool closed, PathCoordinateMode mode)
+void SVGPathBuilder::moveTo(const FloatPoint& targetPoint, PathCoordinateMode mode)
 {
     ASSERT(mode == AbsoluteCoordinates);
-    if (closed && !m_path.isEmpty())
+    if (m_closed && !m_path.isEmpty())
         m_path.closeSubpath();

[pdr., 2015/03/18 19:31:08]

Just call closePath() here?

[fs, 2015/03/19 09:44:09]

I suppose we could do that...

     m_path.moveTo(targetPoint);
+    m_closed = false;
 }
 
 void SVGPathBuilder::lineTo(const FloatPoint& targetPoint, PathCoordinateMode mode)
@@ -51,6 +52,7 @@ void SVGPathBuilder::curveToCubic(const FloatPoint& point1, const FloatPoint& po
 void SVGPathBuilder::closePath()
 {
     m_path.closeSubpath();
+    m_closed = true;
 }
 
 }