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

Unified Diff: test/mjsunit/es6/classes.js

Issue 2430383004: Remove the 'caller' property from the strict-mode arguments map (Closed)
Patch Set: delete an obsolete test Created 4 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
Index: test/mjsunit/es6/classes.js
diff --git a/test/mjsunit/es6/classes.js b/test/mjsunit/es6/classes.js
index fb77dbb8e4b01bb273512b215763dadbb1077661..0261e6e10a78e6d93ce4c1a20ddc3c211c6b1b51 100644
--- a/test/mjsunit/es6/classes.js
+++ b/test/mjsunit/es6/classes.js
@@ -163,15 +163,10 @@
assertThrows('class C extends function B() { with ({}); return B; }() {}',
SyntaxError);
- var D = class extends function() {
- arguments.caller;
caitp 2016/10/27 13:31:02 I don't think we need to remove this line from the
- } {};
+ var D = class extends function() {} {};
assertThrows(function() {
Object.getPrototypeOf(D).arguments;
}, TypeError);
- assertThrows(function() {
- new D;
caitp 2016/10/27 13:31:02 I don't think this should have ever thrown in the
Henrique Ferreiro 2016/10/28 09:56:05 The commit message (d5d15253) says: "Classes: Expa
caitp 2016/10/28 13:00:02 sure, but in this file, it occurs in sloppy mode.
- }, TypeError);
})();

Powered by Google App Engine
This is Rietveld 408576698