Bug 974710 - Stop incorrectly reporting errors with values attribute on the animateMotion element. r=dholbert

content/svg/content/src/SVGMotionSMILAnimationFunction.cpp

@@ -70,17 +70,16 @@ SVGMotionSMILAnimationFunction::SetAttr(nsIAtom* aAttribute,
     if (aParseResult) {
       *aParseResult = rv;
     }
-  } else if (aAttribute == nsGkAtoms::path) {
-    aResult.SetTo(aValue);
-    if (aParseResult) {
-      *aParseResult = NS_OK;
-    }
-    MarkStaleIfAttributeAffectsPath(aAttribute);
-  } else if (aAttribute == nsGkAtoms::by ||
+  } else if (aAttribute == nsGkAtoms::path ||
+             aAttribute == nsGkAtoms::by ||
              aAttribute == nsGkAtoms::from ||
              aAttribute == nsGkAtoms::to ||
              aAttribute == nsGkAtoms::values) {
+    aResult.SetTo(aValue);
     MarkStaleIfAttributeAffectsPath(aAttribute);
+    if (aParseResult) {
+      *aParseResult = NS_OK;
+    }
   } else {
     // Defer to superclass method
     return nsSMILAnimationFunction::SetAttr(aAttribute, aValue,