Need to read the spec properly.

Reviewed by Niko

Correction, 'z' and 'Z' are the only commands that cannot have an extended
list of arguments.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@28575 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 2ea68ae76639a12a3ce321366b6f818c8a9102f7..97e3fa8297d49fc651b5d23b3a25c2533a4119ac 100644 (file)
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,12 @@
+2007-12-09  Oliver Hunt  <oliver@apple.com>
+
+        Reviewed by Niko.
+
+        Somehow I managed to not check in new expected results
+        in the last patch.
+
+        * svg/dom/path-parser-expected.txt:
+
 2007-12-09  Oliver Hunt  <oliver@apple.com>
 
         Reviewed by Niko.

diff --git a/LayoutTests/svg/dom/path-parser-expected.txt b/LayoutTests/svg/dom/path-parser-expected.txt
index 7cd749877ee6e882598cbc1656064169b2c6d381..c2eab41f4187a398fa527ce926069e87049f0daa 100644 (file)
--- a/LayoutTests/svg/dom/path-parser-expected.txt
+++ b/LayoutTests/svg/dom/path-parser-expected.txt
@@ -200,6 +200,7 @@ CONSOLE MESSAGE: line 97: Error: Problem parsing d="M1,1a9,2 6 1 2 2 2,za9 0 2 8
 CONSOLE MESSAGE: line 97: Error: Problem parsing d=""
 CONSOLE MESSAGE: line 97: Error: Problem parsing d="M"
 CONSOLE MESSAGE: line 97: Error: Problem parsing d="M
+CONSOLE MESSAGE: line 97: Error: Problem parsing d="M1,1Z0"
 This test fuzzes the path parser with semi-random attribute values and dumps the results.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 7983bc79bbe92183dd3219c8869f58a527d0d275..1fbf822c79f7f82d445037d91994a16c30de951a 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,13 @@
+2007-12-09  Oliver Hunt  <oliver@apple.com>
+
+        Reviewed by Niko.
+
+        Correction, 'z' and 'Z' are the only commands that cannot have an extended
+        list of arguments.
+
+        * svg/SVGParserUtilities.cpp:
+        (WebCore::SVGPathParser::parseSVG):
+
 2007-12-09  Oliver Hunt  <oliver@apple.com>
 
         Reviewed by Niko.

diff --git a/WebCore/svg/SVGParserUtilities.cpp b/WebCore/svg/SVGParserUtilities.cpp
index 0cfaaf4c829b0ccc20f09541c18144ff7481cce0..85914115ab8c564260d9f3ff92a04edbc09128fd 100644 (file)
--- a/WebCore/svg/SVGParserUtilities.cpp
+++ b/WebCore/svg/SVGParserUtilities.cpp
@@ -513,14 +513,11 @@ bool SVGPathParser::parseSVG(const String& s, bool process)
 
         // Check for remaining coordinates in the current command.
         if ((*ptr == '+' || *ptr == '-' || (*ptr >= '0' && *ptr <= '9')) &&
-            (command != 'z' && command !='a' && command != 'A')) {
+            (command != 'z' && command != 'Z')) {
             if (command == 'M')
                 command = 'L';
             else if (command == 'm')
                 command = 'l';
-            else
-                // Garbage data at the end of the path
-                return false;
         } else
             command = *(ptr++);