Refactor SVGAnimatedLength negative values mode

third_party/WebKit/Source/core/svg/SVGAnimatedLength.cpp

Index: third_party/WebKit/Source/core/svg/SVGAnimatedLength.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGAnimatedLength.cpp b/third_party/WebKit/Source/core/svg/SVGAnimatedLength.cpp
index a22e1f5014faf7a4e62e3fa45c0fcbbe3ef8543b..1ed51ed635d72b1bcbc644430597ba61ee22cb4d 100644
--- a/third_party/WebKit/Source/core/svg/SVGAnimatedLength.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGAnimatedLength.cpp
@@ -33,6 +33,7 @@
 #include "core/svg/SVGAnimatedLength.h"
 
 #include "core/svg/SVGElement.h"
+#include "core/svg/SVGLength.h"
 
 namespace blink {
 
@@ -50,7 +51,7 @@ void SVGAnimatedLength::setBaseValueAsString(const String& value, SVGParsingErro
     if (es.hadException()) {
         parseError = ParsingAttributeFailedError;
         baseValue()->newValueSpecifiedUnits(CSSPrimitiveValue::UnitType::UserUnits, 0);
-    } else if (m_negativeValuesMode == ForbidNegativeLengths && baseValue()->valueInSpecifiedUnits() < 0) {
+    } else if (SVGLength::negativeValuesModeForAnimatedLengthAttribute(attributeName()) == ForbidNegativeLengths && baseValue()->valueInSpecifiedUnits() < 0) {

[fs, 2015/12/02 14:45:42]

It's possible that we ought/need to re-order these condition now (since the
first one is more expensive now), but we can wait for the (perf)bots to come
down with the verdict on that first I think.

         parseError = NegativeValueForbiddenError;
     }
 }