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

Unified Diff: LayoutTests/fast/canvas/script-tests/canvas-getContext-invalid.js

Issue 1249913003: The contextId argument for HTMLCanvasElement getContext() api should not be optional (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 5 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/canvas/script-tests/canvas-getContext-invalid.js
diff --git a/LayoutTests/fast/canvas/script-tests/canvas-getContext-invalid.js b/LayoutTests/fast/canvas/script-tests/canvas-getContext-invalid.js
index b957468524408c17633e729a1b70225080dfd3e0..dec12679ad24d825856905bb1fe137784ae253ed 100644
--- a/LayoutTests/fast/canvas/script-tests/canvas-getContext-invalid.js
+++ b/LayoutTests/fast/canvas/script-tests/canvas-getContext-invalid.js
@@ -8,3 +8,6 @@ shouldBe("canvas.getContext('This is clearly not a valid context name.')", "null
shouldBe("canvas.getContext('2d\0')", "null");
shouldBe("canvas.getContext('2\uFF44')", "null");
shouldBe("canvas.getContext('2D')", "null");
+shouldThrow("canvas.getContext()");
+shouldBe("canvas.getContext('null')", "null");
+shouldBe("canvas.getContext('undefined')", "null");

Powered by Google App Engine
This is Rietveld 408576698