Implement addPath() method for Path object

Source/core/html/canvas/DOMPath.h

Index: Source/core/html/canvas/DOMPath.h
diff --git a/Source/core/html/canvas/DOMPath.h b/Source/core/html/canvas/DOMPath.h
index abb01a6fd71745ea202006276c032113dc686b11..fdbe8b60c5866618332b9b060f1676c14fefdc42 100644
--- a/Source/core/html/canvas/DOMPath.h
+++ b/Source/core/html/canvas/DOMPath.h
@@ -30,6 +30,7 @@
 
 #include "bindings/v8/ScriptWrappable.h"
 #include "core/html/canvas/CanvasPathMethods.h"
+#include "core/svg/SVGMatrix.h"
 #include "core/svg/SVGPathUtilities.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/RefCounted.h"
@@ -47,7 +48,20 @@ public:
 
     const Path& path() const { return m_path; }
 
+    DOMPath(DOMPath* path)

[Justin Novosad, 2014/02/18 16:09:29]

This is weird. This is kind of like a public copy constructor, but not quite,
yet this class has WTF_MAKE_NONCOPYABLE(DOMPath).  It looks too much like
cheating.  Perhaps we should have a clone() method instead?

[pals, 2014/02/19 11:47:22]

Done. Reverted this change.

+        : CanvasPathMethods()
+    {
+        ScriptWrappable::init(this);
+        m_path = path->path();

[rwlbuis, 2014/02/18 15:59:59]

You can put this in the initialization list, it is a bit more efficient.

[pals, 2014/02/19 11:47:22]

Done.

+    }
+
     virtual ~DOMPath() { }
+
+    void addPath(const DOMPath& path, const SVGMatrix& transform)
+    {
+        Path src = path.path();
+        m_path.addPath(src, transform);
+    }
 private:
     DOMPath() : CanvasPathMethods()
     {
@@ -61,13 +75,6 @@ private:
         m_path = path;
     }
 
-    DOMPath(DOMPath* path)
-        : CanvasPathMethods()
-    {
-        ScriptWrappable::init(this);
-        m_path = path->path();
-    }
-
     DOMPath(const String& pathData)
         : CanvasPathMethods()
     {