[SVG2] Allow leading and trailing whitespace in svg attributes

Source/core/svg/SVGLength.cpp

Index: Source/core/svg/SVGLength.cpp
diff --git a/Source/core/svg/SVGLength.cpp b/Source/core/svg/SVGLength.cpp
index 958157f365f2db639f460f33a3855588c09378fe..1cd7668ad5acee9fdc4862d3312d53ea5ba5659a 100644
--- a/Source/core/svg/SVGLength.cpp
+++ b/Source/core/svg/SVGLength.cpp
@@ -76,29 +76,37 @@ SVGLengthType stringToLengthType(const CharType*& ptr, const CharType* end)
     const UChar firstChar = *ptr;
 
     if (++ptr == end)
-        return firstChar == '%' ? LengthTypePercentage : LengthTypeUnknown;
+        return firstChar == '%' ? LengthTypePercentage : (isSVGSpace(firstChar) ? LengthTypeNumber : LengthTypeUnknown);

[pdr., 2014/06/02 14:35:19]

This isn't needed: if the first character is a percent sign, it's not a space.

[fs, 2014/06/03 14:35:37]

Nested ternary expressions FTW? =)

if (firstChar == '%')
  return percentage;
else if (isSVGSpace(firstChar))
  return number;
else
  return confuzzled;

 
     const UChar secondChar = *ptr;
-
-    if (++ptr != end)
-        return LengthTypeUnknown;
-
-    if (firstChar == 'e' && secondChar == 'm')
-        return LengthTypeEMS;
-    if (firstChar == 'e' && secondChar == 'x')
-        return LengthTypeEXS;
-    if (firstChar == 'p' && secondChar == 'x')
-        return LengthTypePX;
-    if (firstChar == 'c' && secondChar == 'm')
-        return LengthTypeCM;
-    if (firstChar == 'm' && secondChar == 'm')
-        return LengthTypeMM;
-    if (firstChar == 'i' && secondChar == 'n')
-        return LengthTypeIN;
-    if (firstChar == 'p' && secondChar == 't')
-        return LengthTypePT;
-    if (firstChar == 'p' && secondChar == 'c')
-        return LengthTypePC;
+    ptr++;
+
+    skipOptionalSVGSpaces(ptr, end);
+
+    if (ptr == end) {
+        if (firstChar == 'e' && secondChar == 'm')
+            return LengthTypeEMS;
+        if (firstChar == 'e' && secondChar == 'x')
+            return LengthTypeEXS;
+        if (firstChar == 'p' && secondChar == 'x')
+            return LengthTypePX;
+        if (firstChar == 'c' && secondChar == 'm')
+            return LengthTypeCM;
+        if (firstChar == 'm' && secondChar == 'm')
+            return LengthTypeMM;
+        if (firstChar == 'i' && secondChar == 'n')
+            return LengthTypeIN;
+        if (firstChar == 'p' && secondChar == 't')
+            return LengthTypePT;
+        if (firstChar == 'p' && secondChar == 'c')
+            return LengthTypePC;
+        if (isSVGSpace(secondChar)) {
+            if (isSVGSpace(firstChar))
+                return LengthTypeNumber;

[pdr., 2014/06/02 14:35:19]

Two spaces are a number?

[fs, 2014/06/03 14:35:37]

More like "two spaces are not a unit".

"42  "

And with this and the above I just want to say that I find this parsing
structure kind of hard to follow w/ the WS-bits. I suppose a structure like:

UChar firstChar = *++ptr;
if (firstChar == '%') {
  type = percentage;
} else if (firstChar == 'e') {
  UChar secondChar = ptr < end ? *++ptr : 0;
  if (secondChar == 'm')
    type = ems;
  ...
  else
    return unknown;
} else ... {
  ...
}

if (!skipOptionalSVGSpaces(ptr, end))
  return type;
return unknown;

might read more easily. (Not necessarily suggesting any change be done though.)

(Another option would be sinking the WS-handling out into the caller, i.e.:

  ...
  type = stringToLengthType(ptr, end);
  ASSERT(ptr <= end);
  if (skipOptionalSVGSpaces(ptr, end))
      return false;
  ASSERT(ptr == end);
  return true;

[still requires a minor change to rewind ptr after if second char is
unrecognized])

+            if (firstChar == '%')
+                return LengthTypePercentage;
+        }
+    }
 
     return LengthTypeUnknown;
 }
@@ -189,7 +197,7 @@ static bool parseValueInternal(const String& string, float& convertedNumber, SVG
     const CharType* ptr = string.getCharacters<CharType>();
     const CharType* end = ptr + string.length();
 
-    if (!parseNumber(ptr, end, convertedNumber, false))
+    if (!parseNumber(ptr, end, convertedNumber, ALLOW_LEADING))
         return false;
 
     type = stringToLengthType(ptr, end);