Extended error reporting for SVGNumber/Point/Rect and related types

third_party/WebKit/Source/core/svg/SVGNumber.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 33 matching lines @@
 {
     return create(m_value);
 }
 
 String SVGNumber::valueAsString() const
 {
     return String::number(m_value);
 }
 
 template<typename CharType>
-bool SVGNumber::parse(const CharType*& ptr, const CharType* end)
+SVGParsingError SVGNumber::parse(const CharType*& ptr, const CharType* end)
 {
-    if (!parseNumber(ptr, end, m_value, AllowLeadingAndTrailingWhitespace)) {
-        m_value = 0;
-        return false;
-    }
-
-    if (ptr != end) {
-        m_value = 0;
-        return false;
-    }
-
-    return true;
+    float value = 0;
+    const CharType* start = ptr;
+    if (!parseNumber(ptr, end, value, AllowLeadingAndTrailingWhitespace))
+        return SVGParsingError(SVGParseStatus::ExpectedNumber, ptr - start);
+    if (ptr != end)
+        return SVGParsingError(SVGParseStatus::TrailingGarbage, ptr - start);
+    m_value = value;
+    return SVGParseStatus::NoError;
 }
 
 SVGParsingError SVGNumber::setValueAsString(const String& string)
 {
-    if (string.isEmpty()) {
-        m_value = 0;
+    m_value = 0;
+
+    if (string.isEmpty())
         return SVGParseStatus::NoError;
-    }
 
-    bool valid = false;
     if (string.is8Bit()) {
         const LChar* ptr = string.characters8();
         const LChar* end = ptr + string.length();
-        valid = parse(ptr, end);
-    } else {
-        const UChar* ptr = string.characters16();
-        const UChar* end = ptr + string.length();
-        valid = parse(ptr, end);
+        return parse(ptr, end);
     }
-
-    if (!valid) {
-        m_value = 0;
-        return SVGParseStatus::ParsingFailed;
-    }
-    return SVGParseStatus::NoError;
+    const UChar* ptr = string.characters16();
+    const UChar* end = ptr + string.length();
+    return parse(ptr, end);
 }
 
 void SVGNumber::add(PassRefPtrWillBeRawPtr<SVGPropertyBase> other, SVGElement*)
 {
     setValue(m_value + toSVGNumber(other)->value());
 }
 
 void SVGNumber::calculateAnimatedValue(SVGAnimationElement* animationElement, float percentage, unsigned repeatCount, PassRefPtrWillBeRawPtr<SVGPropertyBase> from, PassRefPtrWillBeRawPtr<SVGPropertyBase> to, PassRefPtrWillBeRawPtr<SVGPropertyBase> toAtEndOfDuration, SVGElement*)
 {
     ASSERT(animationElement);
@@ skipping 23 matching lines @@
     m_value = 0;
     return SVGParseStatus::ParsingFailed;
 }
 
 SVGNumberAcceptPercentage::SVGNumberAcceptPercentage(float value)
     : SVGNumber(value)
 {
 }
 
 }