[SVGDom] Improve whitespace handling in style parsing

experimental/svg/model/SkSVGDOM.cpp

Index: experimental/svg/model/SkSVGDOM.cpp
diff --git a/experimental/svg/model/SkSVGDOM.cpp b/experimental/svg/model/SkSVGDOM.cpp
index df589230e0daa370a6fd82a999fc1a97466fae28..a1ebedefaff1026c978168138faadb6fd32cf6fe 100644
--- a/experimental/svg/model/SkSVGDOM.cpp
+++ b/experimental/svg/model/SkSVGDOM.cpp
@@ -126,8 +126,8 @@ public:
 
             const char* valueSep = strchr(fPos, ':');
             if (valueSep && valueSep < sep) {
-                name.set(fPos, valueSep - fPos);
-                value.set(valueSep + 1, sep - valueSep - 1);
+                name  = TrimmedString(fPos, valueSep - 1);
+                value = TrimmedString(valueSep + 1, sep - 1);
             }
 
             fPos = *sep ? sep + 1 : nullptr;
@@ -145,6 +145,18 @@ private:
         return sep;
     }
 

[robertphillips, 2016/08/08 12:39:30]

Does this need to be class-scoped? It seems like this would be more generally
useful.

[f(malita), 2016/08/08 13:00:02]

Done/moved out of the class.

+    static SkString TrimmedString(const char* first, const char* last) {
+        SkASSERT(first);
+        SkASSERT(last);
+        SkASSERT(first <= last);
+
+        while (first <= last && *first <= ' ') { first++; }
+        while (first <= last && *last  <= ' ') { last--; }
+
+        SkASSERT(last - first + 1 >= 0);
+        return SkString(first, SkTo<size_t>(last - first + 1));
+    }
+
     const char* fPos;
 };