Refactor propagation of parsing errors for SVG attributes

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

Index: third_party/WebKit/Source/core/svg/SVGFitToViewBox.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGFitToViewBox.cpp b/third_party/WebKit/Source/core/svg/SVGFitToViewBox.cpp
index 0de541a33c87301a1faed584c03f3ed88916d20e..9346b9825e98700a5b243b97c5cbc44e26b226d5 100644
--- a/third_party/WebKit/Source/core/svg/SVGFitToViewBox.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGFitToViewBox.cpp
@@ -38,7 +38,7 @@ public:
         return adoptRefWillBeNoop(new SVGAnimatedViewBoxRect(contextElement));
     }
 
-    void setBaseValueAsString(const String&, SVGParsingError&) override;
+    SVGParsingError setBaseValueAsString(const String&) override;
 
 protected:
     SVGAnimatedViewBoxRect(SVGElement* contextElement)
@@ -47,21 +47,15 @@ protected:
     }
 };
 
-void SVGAnimatedViewBoxRect::setBaseValueAsString(const String& value, SVGParsingError& parseError)
+SVGParsingError SVGAnimatedViewBoxRect::setBaseValueAsString(const String& value)
 {
-    TrackExceptionState es;
+    SVGParsingError parseStatus = baseValue()->setValueAsString(value);
 
-    baseValue()->setValueAsString(value, es);
-
-    if (es.hadException()) {
-        parseError = ParsingAttributeFailedError;
-        return;
-    }
-
-    if (baseValue()->width() < 0 || baseValue()->height() < 0) {
-        parseError = NegativeValueForbiddenError;
+    if (parseStatus == NoError && (baseValue()->width() < 0 || baseValue()->height() < 0)) {
+        parseStatus = NegativeValueForbiddenError;
         baseValue()->setInvalid();
     }
+    return parseStatus;
 }
 
 SVGFitToViewBox::SVGFitToViewBox(SVGElement* element, PropertyMapPolicy propertyMapPolicy)