Bug 602207 - <animateMotion><mpath> uses path's "transform" attribute on <path> elements, which it shouldn't

--HG--
extra : rebase_source : 9c8d0cb8d4c349cacf4b6c1f0db2b74695e74ff0

content/svg/content/src/SVGMotionSMILAnimationFunction.cpp

@@ -255,8 +255,7 @@ SVGMotionSMILAnimationFunction::
       PRBool ok =
         path.GetDistancesFromOriginToEndsOfVisibleSegments(&mPathVertices);
       if (ok && mPathVertices.Length()) {
-        mPath = pathElem->GetFlattenedPath(
-                  pathElem->PrependLocalTransformTo(gfxMatrix()));
+        mPath = pathElem->GetFlattenedPath(gfxMatrix());
       }
     }
   }

layout/reftests/svg/smil/motion/animateMotion-mpath-target-transform-1.svg

@@ -0,0 +1,14 @@
+<svg xmlns="http://www.w3.org/2000/svg"
+     xmlns:xlink="http://www.w3.org/1999/xlink">
+  <title>Test that 'transform' on the &lt;mpath&gt; target has no affect</title>
+  <defs>
+    <path id="path" d="M0,0 L100,100" transform="translate(100,0)"/>
+  </defs>
+  <rect width="100%" height="100%" fill="lime"/>
+  <rect x="10" y="10" width="100" height="100" fill="red"/>
+  <rect x="10" y="10" width="100" height="100" fill="lime">
+    <animateMotion dur="indefinite">
+      <mpath xlink:href="#path"/>
+    </animateMotion>
+  </rect>
+</svg>

layout/reftests/svg/smil/motion/reftest.list

@@ -3,6 +3,7 @@
 
 == animateMotion-by-1.svg      lime.svg
 == animateMotion-from-to-1.svg lime.svg
+== animateMotion-mpath-target-transform-1.svg lime.svg
 == animateMotion-rotate-1.svg  lime.svg
 == animateMotion-rotate-2.svg  lime.svg
 == animateMotion-values-linear-1.svg animateMotion-values-linear-1-ref.svg