Changeset 60708 in webkit for trunk/JavaScriptCore/API/JSCallbackConstructor.cpp

Timestamp:

Jun 4, 2010, 2:38:38 PM (15 years ago)

Author:

[email protected]

Message:

Bug 40187 - Change function signature of NativeConstructor to match NativeFunction

Reviewed by Oliver Hunt.

Mostly for consistency, but constructor & args arguments are redundant,
and this will help if we wish to be able to JIT calls to more constructors.

JavaScriptCore:

• API/JSCallbackConstructor.cpp:

(JSC::constructJSCallback):

• API/JSCallbackObject.h:
• API/JSCallbackObjectFunctions.h:

(JSC::::construct):

• interpreter/Interpreter.cpp:

(JSC::Interpreter::executeConstruct):

• interpreter/Interpreter.h:
• jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

• runtime/ArrayConstructor.cpp:

(JSC::constructWithArrayConstructor):

• runtime/BooleanConstructor.cpp:

(JSC::constructWithBooleanConstructor):

• runtime/ConstructData.cpp:

(JSC::construct):

• runtime/ConstructData.h:
• runtime/DateConstructor.cpp:

(JSC::constructWithDateConstructor):

• runtime/Error.cpp:

(JSC::constructNativeError):
(JSC::Error::create):

• runtime/ErrorConstructor.cpp:

(JSC::constructWithErrorConstructor):

• runtime/FunctionConstructor.cpp:

(JSC::constructWithFunctionConstructor):

• runtime/NativeErrorConstructor.cpp:

(JSC::constructWithNativeErrorConstructor):

• runtime/NativeErrorConstructor.h:

(JSC::NativeErrorConstructor::errorStructure):

• runtime/NumberConstructor.cpp:

(JSC::constructWithNumberConstructor):

• runtime/ObjectConstructor.cpp:

(JSC::constructWithObjectConstructor):

• runtime/RegExpConstructor.cpp:

(JSC::constructWithRegExpConstructor):

• runtime/StringConstructor.cpp:

(JSC::constructWithStringConstructor):

WebCore:

• bindings/js/JSArrayBufferConstructor.cpp:

(WebCore::constructCanvasArrayBuffer):

• bindings/js/JSAudioConstructor.cpp:

(WebCore::constructAudio):

• bindings/js/JSEventSourceConstructor.cpp:

(WebCore::constructEventSource):

• bindings/js/JSFloatArrayConstructor.cpp:

(WebCore::constructCanvasFloatArray):

• bindings/js/JSImageConstructor.cpp:

(WebCore::constructImage):

• bindings/js/JSInt16ArrayConstructor.cpp:

(WebCore::constructCanvasShortArray):

• bindings/js/JSInt32ArrayConstructor.cpp:

(WebCore::constructCanvasIntArray):

• bindings/js/JSInt8ArrayConstructor.cpp:

(WebCore::constructCanvasByteArray):

• bindings/js/JSMessageChannelConstructor.cpp:

(WebCore::JSMessageChannelConstructor::construct):

• bindings/js/JSMessageChannelConstructor.h:
• bindings/js/JSOptionConstructor.cpp:

(WebCore::constructHTMLOptionElement):

• bindings/js/JSSharedWorkerConstructor.cpp:

(WebCore::constructSharedWorker):

• bindings/js/JSUint16ArrayConstructor.cpp:

(WebCore::constructCanvasUnsignedShortArray):

• bindings/js/JSUint32ArrayConstructor.cpp:

(WebCore::constructCanvasUnsignedIntArray):

• bindings/js/JSUint8ArrayConstructor.cpp:

(WebCore::constructCanvasUnsignedByteArray):

• bindings/js/JSWebKitCSSMatrixConstructor.cpp:

(WebCore::constructWebKitCSSMatrix):

• bindings/js/JSWebKitPointConstructor.cpp:

(WebCore::constructWebKitPoint):

• bindings/js/JSWebSocketConstructor.cpp:

(WebCore::constructWebSocket):

• bindings/js/JSWorkerConstructor.cpp:

(WebCore::constructWorker):

• bindings/js/JSXMLHttpRequestConstructor.cpp:

(WebCore::constructXMLHttpRequest):

• bindings/js/JSXSLTProcessorConstructor.cpp:

(WebCore::constructXSLTProcessor):

• bindings/scripts/CodeGeneratorJS.pm:
• bridge/runtime_object.cpp:

(JSC::Bindings::callRuntimeConstructor):

File:

• trunk/JavaScriptCore/API/JSCallbackConstructor.cpp (modified) (3 diffs)

trunk/JavaScriptCore/API/JSCallbackConstructor.cpp

@@ -53 +53 @@
 }
 
-static JSObject* constructJSCallback(ExecState* exec, JSObject* constructor, const ArgList& args)
+static EncodedJSValue JSC_HOST_CALL constructJSCallback(ExecState* exec)
 {
+    JSObject* constructor = exec->callee();
     JSContextRef ctx = toRef(exec);
     JSObjectRef constructorRef = toRef(constructor);
@@ -60 +61 @@
     JSObjectCallAsConstructorCallback callback = static_cast<JSCallbackConstructor*>(constructor)->callback();
     if (callback) {
-        int argumentCount = static_cast<int>(args.size());
+        int argumentCount = static_cast<int>(exec->argumentCount());
         Vector<JSValueRef, 16> arguments(argumentCount);
         for (int i = 0; i < argumentCount; i++)
-            arguments[i] = toRef(exec, args.at(i));
+            arguments[i] = toRef(exec, exec->argument(i));
 
         JSValueRef exception = 0;
@@ -73 +74 @@
         if (exception)
             exec->setException(toJS(exec, exception));
-        return toJS(result);
+        return JSValue::encode(toJS(result));
     }
     
-    return toJS(JSObjectMake(ctx, static_cast<JSCallbackConstructor*>(constructor)->classRef(), 0));
+    return JSValue::encode(toJS(JSObjectMake(ctx, static_cast<JSCallbackConstructor*>(constructor)->classRef(), 0)));
 }