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

Unified Diff: third_party/WebKit/LayoutTests/svg/parser/whitespace-number.html

Issue 2454413002: Break svg/parser/whitespace tests apart some more. (Closed)
Patch Set: Created 4 years, 2 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: third_party/WebKit/LayoutTests/svg/parser/whitespace-number.html
diff --git a/third_party/WebKit/LayoutTests/svg/parser/whitespace-number.html b/third_party/WebKit/LayoutTests/svg/parser/whitespace-number.html
index 6031381cf250d7c648507278a806dacc55fa7c5a..7d752df9ff07028394b8b9d5ddcecce77b2409ef 100644
--- a/third_party/WebKit/LayoutTests/svg/parser/whitespace-number.html
+++ b/third_party/WebKit/LayoutTests/svg/parser/whitespace-number.html
@@ -28,7 +28,6 @@ testType("<number>",
0, // expected default value
whitespace,
[ "-47", ".1", "0.35", "1e-10", "+32", "+17E-1", "17e+2" ], // valid
- [ Number.NaN, Number.POSITIVE_INFINITY, Number.NEGATIVE_INFINITY, "fnord", "E", "e", "e+", "E-", "-", "+", "-.", ".-", ".", "+.", ".E0", "e1" ], // invalid
[ "" ], // valid units
garbage,
function(elm, value) { assert_approx_equals(elm.offset.baseVal, parseFloat(value), EPSILON); },
@@ -40,10 +39,30 @@ testType("<percentage>",
0, // expected default value
whitespace,
[ "-47", ".1", "0.35", "1e-10", "+32", "+17E-1", "17e+2" ], // valid
- [ Number.NaN, Number.POSITIVE_INFINITY, Number.NEGATIVE_INFINITY, "fnord", "E", "e", "e+", "E-", "-", "+", "-.", ".-", ".", "+.", ".E0", "e1" ], // invalid
[ "%" ], // valid units
garbage,
function(elm, value) { assert_approx_equals(elm.offset.baseVal, parseFloat(value)/ 100, EPSILON); },
function(elm, expected) { assert_approx_equals(elm.offset.baseVal, expected, EPSILON); } );
+testInvalidType("<number>",
+ document.querySelector("stop"),
+ "offset",
+ 0, // expected default value
+ whitespace,
+ [ Number.NaN, Number.POSITIVE_INFINITY, Number.NEGATIVE_INFINITY, "fnord", "E", "e", "e+", "E-", "-", "+", "-.", ".-", ".", "+.", ".E0", "e1" ], // invalid
+ [ "" ], // valid units
+ function(elm, value) { assert_approx_equals(elm.offset.baseVal, parseFloat(value), EPSILON); },
+ function(elm, expected) { assert_approx_equals(elm.offset.baseVal, expected, EPSILON); } );
+
+testInvalidType("<percentage>",
+ document.querySelector("stop"),
+ "offset",
+ 0, // expected default value
+ whitespace,
+ [ Number.NaN, Number.POSITIVE_INFINITY, Number.NEGATIVE_INFINITY, "fnord", "E", "e", "e+", "E-", "-", "+", "-.", ".-", ".", "+.", ".E0", "e1" ], // invalid
+ [ "%" ], // valid units
+ function(elm, value) { assert_approx_equals(elm.offset.baseVal, parseFloat(value)/ 100, EPSILON); },
+ function(elm, expected) { assert_approx_equals(elm.offset.baseVal, expected, EPSILON); } );
+
+
</script>

Powered by Google App Engine
This is Rietveld 408576698