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

Unified Diff: test/mjsunit/regress/regress-542099.js

Issue 1396663004: Fix scopes for body of sloppy-mode for-in/of loop (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 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
« src/parser.cc ('K') | « src/parser.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/regress/regress-542099.js
diff --git a/test/message/destructuring-modify-const.js b/test/mjsunit/regress/regress-542099.js
similarity index 50%
copy from test/message/destructuring-modify-const.js
copy to test/mjsunit/regress/regress-542099.js
index cabd924b37ea9f178259f4c979c3b3268920c713..98417a4e48c4e492167c57beebc5979656ce0958 100644
--- a/test/message/destructuring-modify-const.js
+++ b/test/mjsunit/regress/regress-542099.js
@@ -2,8 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-// Flags: --harmony-destructuring
-'use strict';
+// Previously, this caused a CHECK fail in debug mode
+// https://code.google.com/p/chromium/issues/detail?id=542099
-const { x : x, y : y } = { x : 1, y : 2 };
-x++;
+for (foo in []) function foo() { return foo; }
rossberg 2015/10/13 10:52:04 Can you extend this test such that it also tests t
Dan Ehrenberg 2015/10/13 14:58:40 Done
« src/parser.cc ('K') | « src/parser.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698