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

Unified Diff: LayoutTests/svg/custom/script-tests/getSubStringLength.js

Issue 22798002: Make SVGTextContentElement methods behave consistently with Firefox and IE (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix nits Created 7 years, 4 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/custom/script-tests/getSubStringLength.js
diff --git a/LayoutTests/svg/custom/script-tests/getSubStringLength.js b/LayoutTests/svg/custom/script-tests/getSubStringLength.js
index d0fbb57368d9d7ae164650208ce925b804784741..90580fdc553c036484f0fceacfbcfc65c399c7e0 100644
--- a/LayoutTests/svg/custom/script-tests/getSubStringLength.js
+++ b/LayoutTests/svg/custom/script-tests/getSubStringLength.js
@@ -23,10 +23,10 @@ shouldBe("svgText.getSubStringLength(1, 1)", "20");
shouldBe("svgText.getSubStringLength(2, 1)", "20");
shouldBe("svgText.getSubStringLength(0, 3)", "60");
-shouldThrow("svgText.getSubStringLength(1, -1)");
-shouldThrow("svgText.getSubStringLength(2, -1)");
-shouldThrow("svgText.getSubStringLength(3, -1)");
-shouldThrow("svgText.getSubStringLength(3, -3)");
+shouldNotThrow("svgText.getSubStringLength(1, -1)");
+shouldNotThrow("svgText.getSubStringLength(2, -1)");
+shouldThrow("svgText.getSubStringLength(3, -1)", "'IndexSizeError: Index or size was negative, or greater than the allowed value.'");
+shouldThrow("svgText.getSubStringLength(3, -3)", "'IndexSizeError: Index or size was negative, or greater than the allowed value.'");
// cleanup
document.documentElement.removeChild(svgRoot);
« no previous file with comments | « LayoutTests/svg/custom/getSubStringLength-expected.txt ('k') | LayoutTests/svg/custom/script-tests/selectSubString.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698