Refactor propagation of parsing errors for SVG attributes

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

 /*
  * Copyright (C) 2004, 2005, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006 Rob Buis <buis@kde.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 84 matching lines @@
         const CharType* start = ptr;
         while (ptr < end && *ptr != delimiter && !isHTMLSpace<CharType>(*ptr))
             ptr++;
         if (ptr == start)
             break;
         m_values.append(String(start, ptr - start));
         skipOptionalSVGSpacesOrDelimiter(ptr, end, delimiter);
     }
 }
 
-void SVGStringList::setValueAsString(const String& data, ExceptionState&)
+SVGParsingError SVGStringList::setValueAsString(const String& data)
 {
     // FIXME: Add more error checking and reporting.
     m_values.clear();
+
     if (data.isEmpty())
-        return;
+        return NoError;
+
     if (data.is8Bit()) {
         const LChar* ptr = data.characters8();
         const LChar* end = ptr + data.length();
         parseInternal(ptr, end);
     } else {
         const UChar* ptr = data.characters16();
         const UChar* end = ptr + data.length();
         parseInternal(ptr, end);
     }
+    return NoError;
 }
 
 String SVGStringList::valueAsString() const
 {
     StringBuilder builder;
 
     Vector<String>::const_iterator it = m_values.begin();
     Vector<String>::const_iterator itEnd = m_values.end();
     if (it != itEnd) {
         builder.append(*it);
@@ skipping 32 matching lines @@
 
 float SVGStringList::calculateDistance(PassRefPtrWillBeRawPtr<SVGPropertyBase>, SVGElement*)
 {
     // SVGStringList is never animated.
     ASSERT_NOT_REACHED();
 
     return -1.0f;
 }
 
 }