Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(67)

Unified Diff: LayoutTests/svg/dom/script-tests/path-parser.js

Issue 1015433003: Don't handle out-of-range parameters for arcs in UnalteredParsing mode (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/svg/dom/script-tests/path-parser.js
diff --git a/LayoutTests/svg/dom/script-tests/path-parser.js b/LayoutTests/svg/dom/script-tests/path-parser.js
index 52b27263906e34f3aea1aa6af9117a7ac9030cdd..3274242bbc6fcec54b7cdc468c851547c0bc1ea9 100644
--- a/LayoutTests/svg/dom/script-tests/path-parser.js
+++ b/LayoutTests/svg/dom/script-tests/path-parser.js
@@ -100,9 +100,7 @@ shouldBe("parsePath('M100,200 a3,4,5,116,7')", "'M100,200 a3,4,5,1,1,6,7'");
shouldBe("parsePath('M100,200 a3,4,5,2,1,6,7')", "'M100,200'");
shouldBe("parsePath('M100,200 a3,4,5,1,2,6,7')", "'M100,200'");
-// FIXME: This uses 'If rx = 0 or ry = 0 then this arc is treated as a straight line segment (a "lineto") joining the endpoints.'
-// I think the SVG DOM should still show the arc segment, fix that!
-shouldBe("parsePath('M100,200 a0,4,5,0,0,10,0 a4,0,5,0,0,0,10 a0,0,5,0,0,-10,0 z')", "'M100,200 l10,0 l0,10 l-10,0 Z'");
+shouldBe("parsePath('M100,200 a0,4,5,0,0,10,0 a4,0,5,0,0,0,10 a0,0,5,0,0,-10,0 z')", "'M100,200 a0,4,5,0,0,10,0 a4,0,5,0,0,0,10 a0,0,5,0,0,-10,0 Z'");
shouldBe("parsePath('M1,2,3,4')", "'M1,2 L3,4'");
shouldBe("parsePath('m100,200,3,4')", "'m100,200 l3,4'");

Powered by Google App Engine
This is Rietveld 408576698