An empty SVG path string is not in error

Source/core/svg/SVGPathUtilities.cpp

 /*
  * Copyright (C) Research In Motion Limited 2010, 2012. All rights reserved.
  *
  * 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,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 80 matching lines @@
     static SVGPathBlender* s_blender = 0;
     if (!s_blender)
         s_blender = new SVGPathBlender;
 
     return s_blender;
 }
 
 bool buildPathFromString(const String& d, Path& result)
 {
     if (d.isEmpty())
-        return false;
+        return true;
 
     SVGPathBuilder* builder = globalSVGPathBuilder(result);
 
     OwnPtr<SVGPathStringSource> source = SVGPathStringSource::create(d);
     SVGPathParser* parser = globalSVGPathParser(source.get(), builder);
     bool ok = parser->parsePathDataFromSource(NormalizedParsing);
     parser->cleanup();
     return ok;
 }
 
 bool buildPathFromByteStream(const SVGPathByteStream* stream, Path& result)
 {
     ASSERT(stream);
     if (stream->isEmpty())
-        return false;
+        return true;
 
     SVGPathBuilder* builder = globalSVGPathBuilder(result);
 
     SVGPathByteStreamSource source(stream);
     SVGPathParser* parser = globalSVGPathParser(&source, builder);
     bool ok = parser->parsePathDataFromSource(NormalizedParsing);
     parser->cleanup();
     return ok;
 }
 
 bool buildStringFromByteStream(const SVGPathByteStream* stream, String& result, PathParsingMode parsingMode)
 {
     ASSERT(stream);
     if (stream->isEmpty())
-        return false;
+        return true;
 
     SVGPathStringBuilder* builder = globalSVGPathStringBuilder();
 
     SVGPathByteStreamSource source(stream);
     SVGPathParser* parser = globalSVGPathParser(&source, builder);
     bool ok = parser->parsePathDataFromSource(parsingMode);
     result = builder->result();
     parser->cleanup();
     return ok;
 }
 
 bool buildSVGPathByteStreamFromString(const String& d, SVGPathByteStream* result, PathParsingMode parsingMode)
 {
     ASSERT(result);
     result->clear();
     if (d.isEmpty())
-        return false;
+        return true;
 
     // The string length is typically a minor overestimate of eventual byte stream size, so it avoids us a lot of reallocs.
     result->reserveInitialCapacity(d.length());
 
     SVGPathByteStreamBuilder* builder = globalSVGPathByteStreamBuilder(result);
 
     OwnPtr<SVGPathStringSource> source = SVGPathStringSource::create(d);
     SVGPathParser* parser = globalSVGPathParser(source.get(), builder);
     bool ok = parser->parsePathDataFromSource(parsingMode);
     parser->cleanup();
 
     result->shrinkToFit();
 
     return ok;
 }
 
 bool addToSVGPathByteStream(SVGPathByteStream* fromStream, const SVGPathByteStream* byStream, unsigned repeatCount)
 {
     ASSERT(fromStream);
     ASSERT(byStream);
     if (fromStream->isEmpty() || byStream->isEmpty())
-        return false;
+        return true;
 
     SVGPathByteStreamBuilder* builder = globalSVGPathByteStreamBuilder(fromStream);
 
     OwnPtr<SVGPathByteStream> fromStreamCopy = fromStream->copy();
     fromStream->clear();
 
     SVGPathByteStreamSource fromSource(fromStreamCopy.get());
     SVGPathByteStreamSource bySource(byStream);
     SVGPathBlender* blender = globalSVGPathBlender();
     bool ok = blender->addAnimatedPath(&fromSource, &bySource, builder, repeatCount);
@@ skipping 46 matching lines @@
 
     SVGPathByteStreamSource source(stream);
     SVGPathParser* parser = globalSVGPathParser(&source, builder);
     bool ok = parser->parsePathDataFromSource(NormalizedParsing);
     point = builder->currentPoint();
     parser->cleanup();
     return ok;
 }
 
 }