Cantralize the place when we construct the default 'this' object that JSC doesn't construct.

Removes code duplication.

This also indirrectly fixes the QMetaObjectWrapperObject where this was missing

2 files changed, 3 insertions, 27 deletions

diff --git a/src/script/bridge/qscriptclassobject.cpp b/src/script/bridge/qscriptclassobject.cpp
index 3a41dae..9b9af9e 100644
--- a/src/script/bridge/qscriptclassobject.cpp
+++ b/src/script/bridge/qscriptclassobject.cpp
@@ -243,17 +243,9 @@ JSC::JSObject* ClassObjectDelegate::construct(JSC::ExecState *exec, JSC::JSObjec
     QScriptObjectDelegate *delegate = obj->delegate();
     QScriptClass *scriptClass = static_cast<ClassObjectDelegate*>(delegate)->scriptClass();
 
-    JSC::Structure* structure;
-    JSC::JSValue prototype = JSC::asObject(callee)->get(exec, exec->propertyNames().prototype);
-    if (prototype.isObject())
-        structure = JSC::asObject(prototype)->inheritorID();
-    else
-        structure = exec->lexicalGlobalObject()->emptyObjectStructure();
-    JSC::JSObject* thisObject = new (exec) QScriptObject(structure);
-
     QScriptEnginePrivate *eng_p = scriptEngineFromExec(exec);
     JSC::ExecState *oldFrame = eng_p->currentFrame;
-    eng_p->pushContext(exec, thisObject, args, callee, true);
+    eng_p->pushContext(exec, JSC::JSValue(), args, callee, true);
     QScriptContext *ctx = eng_p->contextForFrame(eng_p->currentFrame);
 
     QScriptValue defaultObject = ctx->thisObject();
diff --git a/src/script/bridge/qscriptfunction.cpp b/src/script/bridge/qscriptfunction.cpp
index f3880b4..08f0ca0 100644
--- a/src/script/bridge/qscriptfunction.cpp
+++ b/src/script/bridge/qscriptfunction.cpp
@@ -109,19 +109,11 @@ JSC::JSValue FunctionWrapper::proxyCall(JSC::ExecState *exec, JSC::JSObject *cal
 JSC::JSObject* FunctionWrapper::proxyConstruct(JSC::ExecState *exec, JSC::JSObject *callee,
                                                const JSC::ArgList &args)
 {
-    JSC::Structure* structure;
-    JSC::JSValue prototype = JSC::asObject(callee)->get(exec, exec->propertyNames().prototype);
-    if (prototype.isObject())
-        structure = JSC::asObject(prototype)->inheritorID();
-    else
-        structure = exec->lexicalGlobalObject()->emptyObjectStructure();
-    JSC::JSObject* thisObject = new (exec) QScriptObject(structure);
-
     FunctionWrapper *self = static_cast<FunctionWrapper*>(callee);
     QScriptEnginePrivate *eng_p = QScript::scriptEngineFromExec(exec);
 
     JSC::ExecState *oldFrame = eng_p->currentFrame;
-    eng_p->pushContext(exec, thisObject, args, callee, true);
+    eng_p->pushContext(exec, JSC::JSValue(), args, callee, true);
     QScriptContext *ctx = eng_p->contextForFrame(eng_p->currentFrame);
 
     QScriptValue result = self->data->function(ctx, QScriptEnginePrivate::get(eng_p));
@@ -179,19 +171,11 @@ JSC::JSValue FunctionWithArgWrapper::proxyCall(JSC::ExecState *exec, JSC::JSObje
 JSC::JSObject* FunctionWithArgWrapper::proxyConstruct(JSC::ExecState *exec, JSC::JSObject *callee,
                                                       const JSC::ArgList &args)
 {
-    JSC::Structure* structure;
-    JSC::JSValue prototype = JSC::asObject(callee)->get(exec, exec->propertyNames().prototype);
-    if (prototype.isObject())
-        structure = JSC::asObject(prototype)->inheritorID();
-    else
-        structure = exec->lexicalGlobalObject()->emptyObjectStructure();
-    JSC::JSObject* thisObject = new (exec) QScriptObject(structure);
-
     FunctionWithArgWrapper *self = static_cast<FunctionWithArgWrapper*>(callee);
     QScriptEnginePrivate *eng_p = QScript::scriptEngineFromExec(exec);
 
     JSC::ExecState *oldFrame = eng_p->currentFrame;
-    eng_p->pushContext(exec, thisObject, args, callee, true);
+    eng_p->pushContext(exec, JSC::JSValue(), args, callee, true);
     QScriptContext *ctx = eng_p->contextForFrame(eng_p->currentFrame);
 
     QScriptValue result = self->data->function(ctx, QScriptEnginePrivate::get(eng_p) , self->data->arg);