Extended error reporting for SVG path parsing

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

Index: third_party/WebKit/Source/core/svg/SVGParsingError.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGParsingError.cpp b/third_party/WebKit/Source/core/svg/SVGParsingError.cpp
index da2345ddc9f97f47dccb3a273603c990d72bf5a7..9602b5ffaffba4f244cf17c3b34b44bc964b84dd 100644
--- a/third_party/WebKit/Source/core/svg/SVGParsingError.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGParsingError.cpp
@@ -28,14 +28,20 @@ std::pair<const char*, const char*> messageForStatus(SVGParseStatus status)
     switch (status) {
     case SVGParseStatus::TrailingGarbage:
         return std::make_pair("Trailing garbage, ", ".");
+    case SVGParseStatus::ExpectedArcFlag:
+        return std::make_pair("Expected path flag ('0' or '1'), ", ".");

[f(malita), 2016/01/27 14:51:13]

Nit: "Expected arc flag"? Or maybe you're planning to use this for other flags
too - then maybe change the enum to ExpectedPathFlag?

[fs, 2016/01/27 14:57:56]

Yes, I think I had intended to say "arc flag" here... Will fix.

     case SVGParseStatus::ExpectedBoolean:
         return std::make_pair("Expected 'true' or 'false', ", ".");
     case SVGParseStatus::ExpectedEnumeration:
         return std::make_pair("Unrecognized enumerated value, ", ".");
     case SVGParseStatus::ExpectedLength:
         return std::make_pair("Expected length, ", ".");
+    case SVGParseStatus::ExpectedMoveToCommand:
+        return std::make_pair("Expected moveto path command ('M' or 'm'), ", ".");
     case SVGParseStatus::ExpectedNumber:
         return std::make_pair("Expected number, ", ".");
+    case SVGParseStatus::ExpectedPathCommand:
+        return std::make_pair("Expected path command, ", ".");
     case SVGParseStatus::NegativeValue:
         return std::make_pair("A negative value is not valid. (", ")");
     case SVGParseStatus::ParsingFailed: