LibJS: Set the bound |this| value to the |this| value of the current

scope for arrow functions

Libraries/LibJS/Runtime/ScriptFunction.cpp

@@ -53,7 +53,7 @@ ScriptFunction* ScriptFunction::create(GlobalObject& global_object, const FlyStr
 }
 
 ScriptFunction::ScriptFunction(const FlyString& name, const Statement& body, Vector<FunctionNode::Parameter> parameters, i32 m_function_length, LexicalEnvironment* parent_environment, Object& prototype, bool is_arrow_function)
-    : Function(prototype)
+    : Function(prototype, is_arrow_function ? interpreter().this_value() : Value(), {})
     , m_name(name)
     , m_body(body)
     , m_parameters(move(parameters))

Libraries/LibJS/Tests/Function.prototype.bind.js

@@ -102,9 +102,8 @@ try {
     //     assert(strictIdentity.bind(undefined)() === undefined);
     // })();
 
-    // FIXME: Uncomment me when arrow functions have the correct |this| value.
+    // Arrow functions can not have their |this| value set.
-    // // Arrow functions can not have their |this| value set.
+    assert((() => this).bind("foo")() === globalThis)
-    // assert((() => this).bind("foo")() === globalThis)
 
     console.log("PASS");
 } catch (e) {

Libraries/LibJS/Tests/arrow-functions.js

@@ -64,6 +64,19 @@ try {
     assert(foo === undefined);
     assert(bar === undefined);
 
+    function FooBar() {
+      this.x = {
+        y: () => this,
+        z: function () {
+          return (() => this)();
+        }
+      };
+    }
+
+    var foobar = new FooBar();
+    assert(foobar.x.y() === foobar);
+    assert(foobar.x.z() === foobar.x);
+
     (() => {
         "use strict";
         assert(isStrictMode());