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

Unified Diff: test/mjsunit/strict-mode.js

Issue 2430383004: Remove the 'caller' property from the strict-mode arguments map (Closed)
Patch Set: fix restored test and other Created 4 years, 1 month 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/strict-mode.js
diff --git a/test/mjsunit/strict-mode.js b/test/mjsunit/strict-mode.js
index f7f95da88c99592e792e9e6b45b13d2ce4b1c545..c45169a7757fb8e77785c9a350b58e6aefef25d7 100644
--- a/test/mjsunit/strict-mode.js
+++ b/test/mjsunit/strict-mode.js
@@ -1111,14 +1111,14 @@ function CheckArgumentsPillDescriptor(func, name) {
}
var args = strict();
- CheckArgumentsPillDescriptor(args, "caller");
+ assertEquals(args.caller, undefined);
adamk 2016/10/31 18:05:36 The proper order for asserts is assertWhatever(ex
CheckArgumentsPillDescriptor(args, "callee");
args = strict(17, "value", strict);
assertEquals(17, args[0])
assertEquals("value", args[1])
assertEquals(strict, args[2]);
- CheckArgumentsPillDescriptor(args, "caller");
+ assertEquals(args.caller, undefined);
CheckArgumentsPillDescriptor(args, "callee");
function outer() {
@@ -1130,14 +1130,14 @@ function CheckArgumentsPillDescriptor(func, name) {
}
var args = outer()();
- CheckArgumentsPillDescriptor(args, "caller");
+ assertEquals(args.caller, undefined);
CheckArgumentsPillDescriptor(args, "callee");
args = outer()(17, "value", strict);
assertEquals(17, args[0])
assertEquals("value", args[1])
assertEquals(strict, args[2]);
- CheckArgumentsPillDescriptor(args, "caller");
+ assertEquals(args.caller, undefined);
CheckArgumentsPillDescriptor(args, "callee");
})();

Powered by Google App Engine
This is Rietveld 408576698