v8 api: Correctly set up the prototype property for constructors

Also remove some debug output for methods that are actually working
as intended.

Change-Id: Id5df0c87eda40f8ae9769cc43fa05a9ecf67c31c
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>

1 files changed, 5 insertions, 4 deletions

diff --git a/src/v4/qv4v8.cpp b/src/v4/qv4v8.cpp
index 6040c9fa9f..0c30df4e49 100644
--- a/src/v4/qv4v8.cpp
+++ b/src/v4/qv4v8.cpp
@@ -1664,7 +1664,7 @@ Local<Function> FunctionTemplate::GetFunction()
     VM::ExecutionEngine *engine = currentEngine();
     VM::Object *o = new (engine->memoryManager) V4V8Function(engine, this);
     VM::Object *proto = new (engine->memoryManager) V4V8Object<VM::FunctionPrototype>(engine, m_prototypeTemplate.get());
-    o->prototype = proto;
+    o->put(engine->current, engine->id_prototype, VM::Value::fromObject(proto));
     return Local<Function>::New(Value::fromVmValue(VM::Value::fromObject(o)));
 }
 
@@ -1759,13 +1759,14 @@ void ObjectTemplate::SetInternalFieldCount(int value)
 
 bool ObjectTemplate::HasExternalResource()
 {
-    Q_UNIMPLEMENTED();
-    Q_UNREACHABLE();
+    // we always reserve the space for the external resource
+    return true;
 }
 
 void ObjectTemplate::SetHasExternalResource(bool value)
 {
-    Q_UNIMPLEMENTED();
+    // no need for this, we always reserve the space for the external resource
+    Q_UNUSED(value);
 }
 
 void ObjectTemplate::MarkAsUseUserObjectComparison()