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

Unified Diff: third_party/WebKit/LayoutTests/fast/js/mozilla/strict/script-tests/13.1.js

Issue 2318933007: Fix layout test for redeclaration of arguments. (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « third_party/WebKit/LayoutTests/fast/js/mozilla/strict/13.1-expected.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/fast/js/mozilla/strict/script-tests/13.1.js
diff --git a/third_party/WebKit/LayoutTests/fast/js/mozilla/strict/script-tests/13.1.js b/third_party/WebKit/LayoutTests/fast/js/mozilla/strict/script-tests/13.1.js
index 77d5c4468b881088e617af871003ef9276c843e3..2076138410e6b35277a89407ef0b663aa3fb207d 100644
--- a/third_party/WebKit/LayoutTests/fast/js/mozilla/strict/script-tests/13.1.js
+++ b/third_party/WebKit/LayoutTests/fast/js/mozilla/strict/script-tests/13.1.js
@@ -229,15 +229,12 @@ assertEq(testLenientAndStrict('function f(arguments){}',
parsesSuccessfully,
parseRaisesException(SyntaxError)),
true);
-// BUG(v8:4577): In sloppy mode, we currently throw when destructuring
-// parameters include 'arguments', even if a direct argument named
-// 'arguments' is accepted, once the --harmony-sloppy flag is shipped.
assertEq(testLenientAndStrict('function f([arguments]){}',
- parseRaisesException(SyntaxError),
+ parsesSuccessfully,
parseRaisesException(SyntaxError)),
true);
assertEq(testLenientAndStrict('function f({x:arguments}){}',
- parseRaisesException(SyntaxError),
+ parsesSuccessfully,
parseRaisesException(SyntaxError)),
true);
assertEq(testLenientAndStrict('function arguments(){}',
@@ -267,11 +264,11 @@ assertEq(testLenientAndStrict('(function f(arguments){})',
parseRaisesException(SyntaxError)),
true);
assertEq(testLenientAndStrict('(function f([arguments]){})',
- parseRaisesException(SyntaxError),
+ parsesSuccessfully,
parseRaisesException(SyntaxError)),
true);
assertEq(testLenientAndStrict('(function f({x:arguments}){})',
- parseRaisesException(SyntaxError),
+ parsesSuccessfully,
parseRaisesException(SyntaxError)),
true);
assertEq(testLenientAndStrict('(function arguments(){})',
@@ -321,11 +318,11 @@ assertEq(testLenientAndStrict('({set x(arguments){}})',
parseRaisesException(SyntaxError)),
true);
assertEq(testLenientAndStrict('({set x([arguments]){}})',
- parseRaisesException(SyntaxError),
+ parsesSuccessfully,
parseRaisesException(SyntaxError)),
true);
assertEq(testLenientAndStrict('({set x({x:arguments}){}})',
- parseRaisesException(SyntaxError),
+ parsesSuccessfully,
parseRaisesException(SyntaxError)),
true);
assertEq(testLenientAndStrict('({set x(arguments){"use strict";}})',
« no previous file with comments | « third_party/WebKit/LayoutTests/fast/js/mozilla/strict/13.1-expected.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698