Changeset 95751 in webkit for trunk/Source/JavaScriptCore/runtime/JSGlobalObject.cpp

Timestamp:

Sep 22, 2011, 2:22:17 PM (14 years ago)

Author:

[email protected]

Message:

Implement Function.prototype.bind
​https://bugs.webkit.org/show_bug.cgi?id=26382

Reviewed by Sam Weinig.

Source/JavaScriptCore:

This patch provides a basic functional implementation
for Function.bind. It should (hopefully!) be fully
functionally correct, and the bound functions can be
called to quickly (since they are a subclass of
JSFunction, not InternalFunction), but we'll probably
want to follow up with some optimization work to keep
bound calls in JIT code.

• JavaScriptCore.JSVALUE32_64only.exp:
• JavaScriptCore.JSVALUE64only.exp:
• JavaScriptCore.exp:
• JavaScriptCore.xcodeproj/project.pbxproj:
• jit/JITStubs.cpp:

(JSC::JITThunks::hostFunctionStub):

• jit/JITStubs.h:
• jsc.cpp:

(GlobalObject::addFunction):

• runtime/CommonIdentifiers.h:
• runtime/ConstructData.h:
• runtime/Executable.h:

(JSC::NativeExecutable::NativeExecutable):

• runtime/FunctionPrototype.cpp:

(JSC::FunctionPrototype::addFunctionProperties):
(JSC::functionProtoFuncBind):

• runtime/FunctionPrototype.h:
• runtime/JSBoundFunction.cpp: Added.

(JSC::boundFunctionCall):
(JSC::boundFunctionConstruct):
(JSC::JSBoundFunction::create):
(JSC::JSBoundFunction::hasInstance):
(JSC::JSBoundFunction::getOwnPropertySlot):
(JSC::JSBoundFunction::getOwnPropertyDescriptor):
(JSC::JSBoundFunction::JSBoundFunction):
(JSC::JSBoundFunction::finishCreation):

• runtime/JSBoundFunction.h: Added.

(JSC::JSBoundFunction::targetFunction):
(JSC::JSBoundFunction::boundThis):
(JSC::JSBoundFunction::boundArgs):
(JSC::JSBoundFunction::createStructure):

• runtime/JSFunction.cpp:

(JSC::JSFunction::create):
(JSC::JSFunction::finishCreation):
(JSC::createDescriptorForThrowingProperty):
(JSC::JSFunction::getOwnPropertySlot):

• runtime/JSFunction.h:
• runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::getHostFunction):

• runtime/JSGlobalData.h:
• runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::reset):
(JSC::JSGlobalObject::visitChildren):

• runtime/JSGlobalObject.h:

(JSC::JSGlobalObject::boundFunctionStructure):

• runtime/Lookup.cpp:

(JSC::setUpStaticFunctionSlot):

Source/WebCore:

Test: fast/js/function-bind.html

• bindings/js/JSDOMBinding.cpp:

(WebCore::objectToStringFunctionGetter):

• bindings/js/JSDOMWindowCustom.cpp:

(WebCore::nonCachingStaticFunctionGetter):

• bindings/js/JSHistoryCustom.cpp:

(WebCore::nonCachingStaticBackFunctionGetter):
(WebCore::nonCachingStaticForwardFunctionGetter):
(WebCore::nonCachingStaticGoFunctionGetter):

• bindings/js/JSLocationCustom.cpp:

(WebCore::nonCachingStaticReplaceFunctionGetter):
(WebCore::nonCachingStaticReloadFunctionGetter):
(WebCore::nonCachingStaticAssignFunctionGetter):

• Function::create no longer requires functionStructure() to be passed.

LayoutTests:

We now pass Function.bind tests.

• fast/js/Object-getOwnPropertyNames-expected.txt:
• fast/js/basic-strict-mode-expected.txt:
• fast/js/function-bind-expected.txt: Added.
• fast/js/function-bind.html: Added.
• fast/js/mozilla/strict/15.3.4.5-expected.txt:
• fast/js/script-tests/function-bind.js: Added.
• ietestcenter/Javascript/15.2.3.3-4-38-expected.txt:
• ietestcenter/Javascript/15.3.4.5-0-1-expected.txt:
• ietestcenter/Javascript/15.3.4.5-0-2-expected.txt:
• ietestcenter/Javascript/15.3.4.5-13.b-1-expected.txt:
• ietestcenter/Javascript/15.3.4.5-13.b-2-expected.txt:
• ietestcenter/Javascript/15.3.4.5-13.b-3-expected.txt:
• ietestcenter/Javascript/15.3.4.5-13.b-4-expected.txt:
• ietestcenter/Javascript/15.3.4.5-13.b-5-expected.txt:
• ietestcenter/Javascript/15.3.4.5-15-1-expected.txt:
• ietestcenter/Javascript/15.3.4.5-15-2-expected.txt:
• ietestcenter/Javascript/15.3.4.5-16-1-expected.txt:
• ietestcenter/Javascript/15.3.4.5-2-1-expected.txt:
• ietestcenter/Javascript/15.3.4.5-2-2-expected.txt:
• ietestcenter/Javascript/15.3.4.5-2-3-expected.txt:
• ietestcenter/Javascript/15.3.4.5-2-4-expected.txt:
• ietestcenter/Javascript/15.3.4.5-2-5-expected.txt:
• ietestcenter/Javascript/15.3.4.5-2-6-expected.txt:
• ietestcenter/Javascript/15.3.4.5-2-7-expected.txt:
• ietestcenter/Javascript/15.3.4.5-2-8-expected.txt:
• ietestcenter/Javascript/15.3.4.5-2-9-expected.txt:
• ietestcenter/Javascript/15.3.4.5-8-1-expected.txt:
• ietestcenter/Javascript/15.3.4.5-8-2-expected.txt:
• ietestcenter/Javascript/15.3.4.5-9-1-expected.txt:
• ietestcenter/Javascript/15.3.4.5-9-2-expected.txt:

File:

• trunk/Source/JavaScriptCore/runtime/JSGlobalObject.cpp (modified) (4 diffs)

trunk/Source/JavaScriptCore/runtime/JSGlobalObject.cpp

@@ -47 +47 @@
 #include "FunctionConstructor.h"
 #include "FunctionPrototype.h"
+#include "JSBoundFunction.h"
 #include "JSFunction.h"
 #include "JSGlobalObjectFunctions.h"
@@ -188 +189 @@
     m_functionPrototype.set(exec->globalData(), this, FunctionPrototype::create(exec, this, FunctionPrototype::createStructure(exec->globalData(), this, jsNull()))); // The real prototype will be set once ObjectPrototype is created.
     m_functionStructure.set(exec->globalData(), this, JSFunction::createStructure(exec->globalData(), this, m_functionPrototype.get()));
+    m_boundFunctionStructure.set(exec->globalData(), this, JSBoundFunction::createStructure(exec->globalData(), this, m_functionPrototype.get()));
     m_namedFunctionStructure.set(exec->globalData(), this, Structure::addPropertyTransition(exec->globalData(), m_functionStructure.get(), exec->globalData().propertyNames->name, DontDelete | ReadOnly | DontEnum, 0, m_functionNameOffset));
     m_internalFunctionStructure.set(exec->globalData(), this, InternalFunction::createStructure(exec->globalData(), this, m_functionPrototype.get()));
     JSFunction* callFunction = 0;
     JSFunction* applyFunction = 0;
-    m_functionPrototype->addFunctionProperties(exec, this, m_functionStructure.get(), &callFunction, &applyFunction);
+    m_functionPrototype->addFunctionProperties(exec, this, &callFunction, &applyFunction);
     m_callFunction.set(exec->globalData(), this, callFunction);
     m_applyFunction.set(exec->globalData(), this, applyFunction);
@@ -281 +283 @@
     putDirectWithoutTransition(exec->globalData(), Identifier(exec, "URIError"), m_URIErrorConstructor.get(), DontEnum);
 
-    m_evalFunction.set(exec->globalData(), this, JSFunction::create(exec, this, m_functionStructure.get(), 1, exec->propertyNames().eval, globalFuncEval));
+    m_evalFunction.set(exec->globalData(), this, JSFunction::create(exec, this, 1, exec->propertyNames().eval, globalFuncEval));
     putDirectWithoutTransition(exec->globalData(), exec->propertyNames().eval, m_evalFunction.get(), DontEnum);
 
@@ -350 +352 @@
     visitIfNeeded(visitor, &m_errorStructure);
     visitIfNeeded(visitor, &m_functionStructure);
+    visitIfNeeded(visitor, &m_boundFunctionStructure);
     visitIfNeeded(visitor, &m_namedFunctionStructure);
     visitIfNeeded(visitor, &m_numberObjectStructure);