Refactor propagation of parsing errors for SVG attributes

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

 /*
  * Copyright (C) 2014 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 55 matching lines @@
 
 String SVGNumberOptionalNumber::valueAsString() const
 {
     if (m_firstNumber->value() == m_secondNumber->value()) {
         return String::number(m_firstNumber->value());
     }
 
     return String::number(m_firstNumber->value()) + " " + String::number(m_secondNumber->value());
 }
 
-void SVGNumberOptionalNumber::setValueAsString(const String& value, ExceptionState& exceptionState)
+SVGParsingError SVGNumberOptionalNumber::setValueAsString(const String& value)
 {
     float x, y;
+    SVGParsingError parseStatus = NoError;
     if (!parseNumberOptionalNumber(value, x, y)) {
-        exceptionState.throwDOMException(SyntaxError, "The value provided ('" + value + "') is invalid.");
+        parseStatus = ParsingAttributeFailedError;
         x = y = 0;
     }
 
     m_firstNumber->setValue(x);
     m_secondNumber->setValue(y);
+    return parseStatus;
 }
 
 void SVGNumberOptionalNumber::add(PassRefPtrWillBeRawPtr<SVGPropertyBase> other, SVGElement*)
 {
     RefPtrWillBeRawPtr<SVGNumberOptionalNumber> otherNumberOptionalNumber = toSVGNumberOptionalNumber(other);
 
     m_firstNumber->setValue(m_firstNumber->value() + otherNumberOptionalNumber->m_firstNumber->value());
     m_secondNumber->setValue(m_secondNumber->value() + otherNumberOptionalNumber->m_secondNumber->value());
 }
 
@@ skipping 13 matching lines @@
     m_secondNumber->setValue(y);
 }
 
 float SVGNumberOptionalNumber::calculateDistance(PassRefPtrWillBeRawPtr<SVGPropertyBase> other, SVGElement*)
 {
     // FIXME: Distance calculation is not possible for SVGNumberOptionalNumber right now. We need the distance for every single value.
     return -1;
 }
 
 }