Chromium Code Reviews

Unified Diff: third_party/WebKit/LayoutTests/fast/js/function-length.html

Issue 2755773002: Make Window#open arguments optional to match the spec
Patch Set: Updated layout tests for window.open Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
« third_party/WebKit/LayoutTests/external/wpt/html/browsers/the-window-object/window-open.html ('K') | « third_party/WebKit/LayoutTests/external/wpt/html/browsers/the-window-object/window-open.html ('k') | third_party/WebKit/LayoutTests/fast/js/function-length-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/fast/js/function-length.html
diff --git a/third_party/WebKit/LayoutTests/fast/js/function-length.html b/third_party/WebKit/LayoutTests/fast/js/function-length.html
index 147d5aba48ad5f09c5d9950c1a561912b58f165e..b51ca901a6afd6d764a22bfc6e6430cd2a20c992 100644
--- a/third_party/WebKit/LayoutTests/fast/js/function-length.html
+++ b/third_party/WebKit/LayoutTests/fast/js/function-length.html
@@ -8,7 +8,7 @@
description("This tests the length property of functions.");
shouldBe('window.confirm.length', '0');
-shouldBe('window.open.length', '2');
+shouldBe('window.open.length', '0');
shouldBe('window.setTimeout.length', '1');
shouldBe('window.clearTimeout.length', '0');
shouldBe('window.addEventListener.length', '2');
« third_party/WebKit/LayoutTests/external/wpt/html/browsers/the-window-object/window-open.html ('K') | « third_party/WebKit/LayoutTests/external/wpt/html/browsers/the-window-object/window-open.html ('k') | third_party/WebKit/LayoutTests/fast/js/function-length-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine