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

Unified Diff: LayoutTests/fast/shapes/parsing/parsing-test-utils.js

Issue 203413006: [CSS Shapes] shape-outside: ellipse(50% 50% at) causes crash (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 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/fast/shapes/parsing/parsing-test-utils.js
diff --git a/LayoutTests/fast/shapes/parsing/parsing-test-utils.js b/LayoutTests/fast/shapes/parsing/parsing-test-utils.js
index 7379b164c92977a7d2b577d9a9effcd11c680fc8..84efceacd7f9838b6ec7671219c248ff5e28228e 100644
--- a/LayoutTests/fast/shapes/parsing/parsing-test-utils.js
+++ b/LayoutTests/fast/shapes/parsing/parsing-test-utils.js
@@ -146,6 +146,8 @@ var invalidShapeValues = [
"circle(at center center 10px)",
"circle(at 10px 10px closest-side)",
+ "circle(10px at)",
+ "circle(at)",
"ellipse(10px, 20px)", // FIXME: Remove this test once we do not support the deprecated CSS Shapes syntax anymore.
"ellipse(10px, 20px, 30px)", // FIXME: Remove this test once we do not support the deprecated CSS Shapes syntax anymore.
@@ -159,6 +161,8 @@ var invalidShapeValues = [
"ellipse(at 10px 10px 10px)",
"ellipse(at 10px 10px at center)",
"ellipse(at center center 10px)",
+ "ellipse(10px 20px at)",
+ "ellipse(at)",
"polygon()",
"polygon(evenodd 10px 20px, 30px 40px, 40px 50px)",
« no previous file with comments | « LayoutTests/fast/shapes/parsing/parsing-shape-outside-expected.txt ('k') | Source/core/css/parser/CSSPropertyParser.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698