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

Unified Diff: test/mjsunit/harmony/arrow-functions-this.js

Issue 1097283003: Resolve references to "this" the same way as normal variables (Closed) Base URL: https://chromium.googlesource.com/v8/v8@master
Patch Set: Add TODO to fix fat-fingered "this" scoping in script context Created 5 years, 7 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') | « test/mjsunit/debug-scopes.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/harmony/arrow-functions-this.js
diff --git a/test/mjsunit/harmony/arrow-functions-this.js b/test/mjsunit/harmony/arrow-functions-this.js
new file mode 100644
index 0000000000000000000000000000000000000000..6a29a00ea9f90c841acf94448a27faf46bbfc061
--- /dev/null
+++ b/test/mjsunit/harmony/arrow-functions-this.js
@@ -0,0 +1,69 @@
+// Copyright 2015 the V8 project authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// Flags: --harmony-arrow-functions
+
+var object = {};
+var global = this;
+var call = Function.call.bind(Function.call);
+
+var globalSloppyArrow = ()=>this;
arv (Not doing code reviews) 2015/05/05 14:59:10 no aligning of = whitespace around => var global
wingo 2015/05/05 15:49:29 Done.
+var globalStrictArrow = ()=>{ "use strict"; return this; };
+var globalSloppyArrowEval = (s)=>eval(s);
+var globalStrictArrowEval = (s)=>{ "use strict"; return eval(s); };
+
+var sloppyFunctionArrow = function(){ return (()=>this)(); };
+var strictFunctionArrow = function(){ "use strict";
arv (Not doing code reviews) 2015/05/05 14:59:10 indent blocks two spaces whitespace between ) and
wingo 2015/05/05 15:49:29 Done.
+ return (()=>this)(); };
+var sloppyFunctionEvalArrow = function(){ return eval("(()=>this)()"); };
+var strictFunctionEvalArrow = function(){ "use strict";
+ return eval("(()=>this)()"); };
+var sloppyFunctionArrowEval = function(s) { return (()=>eval(s))(); };
+var strictFunctionArrowEval = function(s) { "use strict";
+ return (()=>eval(s))(); };
+
+var withObject = { 'this': object }
+var arrowInsideWith, arrowInsideWithEval;
+with (withObject) {
+ arrowInsideWith = ()=>this;
+ arrowInsideWithEval = (s)=>eval(s);
+}
+
+assertEquals(global, call(globalSloppyArrow, object));
+assertEquals(global, call(globalStrictArrow, object));
+assertEquals(global, call(globalSloppyArrowEval, object, "this"));
+assertEquals(global, call(globalStrictArrowEval, object, "this"));
+assertEquals(global, call(globalSloppyArrowEval, object, "(()=>this)()"));
+assertEquals(global, call(globalStrictArrowEval, object, "(()=>this)()"));
+
+assertEquals(object, call(sloppyFunctionArrow, object));
+assertEquals(global, call(sloppyFunctionArrow, undefined));
+assertEquals(object, call(strictFunctionArrow, object));
+assertEquals(undefined, call(strictFunctionArrow, undefined));
+
+assertEquals(object, call(sloppyFunctionEvalArrow, object));
+assertEquals(global, call(sloppyFunctionEvalArrow, undefined));
+assertEquals(object, call(strictFunctionEvalArrow, object));
+assertEquals(undefined, call(strictFunctionEvalArrow, undefined));
+
+assertEquals(object, call(sloppyFunctionArrowEval, object, "this"));
+assertEquals(global, call(sloppyFunctionArrowEval, undefined, "this"));
+assertEquals(object, call(strictFunctionArrowEval, object, "this"));
+assertEquals(undefined, call(strictFunctionArrowEval, undefined, "this"));
+
+assertEquals(object,
+ call(sloppyFunctionArrowEval, object, "(()=>this)()"));
+assertEquals(global,
+ call(sloppyFunctionArrowEval, undefined, "(()=>this)()"));
+assertEquals(object,
+ call(strictFunctionArrowEval, object, "(()=>this)()"));
+assertEquals(undefined,
+ call(strictFunctionArrowEval, undefined, "(()=>this)()"));
+
+assertEquals(global, call(arrowInsideWith, undefined));
+assertEquals(global, call(arrowInsideWith, object));
+assertEquals(global, call(arrowInsideWithEval, undefined, "this"));
+assertEquals(global, call(arrowInsideWithEval, object, "this"));
+assertEquals(global, call(arrowInsideWithEval, undefined, "(()=>this)()"));
+assertEquals(global, call(arrowInsideWithEval, object, "(()=>this)()"));
« src/parser.cc ('K') | « test/mjsunit/debug-scopes.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698