diff options
Diffstat (limited to 'src/qml/qml')
-rw-r--r-- | src/qml/qml/qqmlvme.cpp | 1 | ||||
-rw-r--r-- | src/qml/qml/v8/qv8contextwrapper.cpp | 21 | ||||
-rw-r--r-- | src/qml/qml/v8/qv8contextwrapper_p.h | 5 |
3 files changed, 19 insertions, 8 deletions
diff --git a/src/qml/qml/qqmlvme.cpp b/src/qml/qml/qqmlvme.cpp index da918c3c71..9db3ee5a4d 100644 --- a/src/qml/qml/qqmlvme.cpp +++ b/src/qml/qml/qqmlvme.cpp @@ -1244,6 +1244,7 @@ v8::Persistent<v8::Object> QQmlVME::run(QQmlContextData *parentCtxt, QQmlScriptD script->initialize(parentCtxt->engine); v8::Local<v8::Object> qmlglobal = v8engine->qmlScope(ctxt, 0); + v8engine->contextWrapper()->takeContextOwnership(qmlglobal); if (!script->m_program.IsEmpty()) { script->m_program->Run(qmlglobal); diff --git a/src/qml/qml/v8/qv8contextwrapper.cpp b/src/qml/qml/v8/qv8contextwrapper.cpp index 9f18afc5cb..bc64189cbd 100644 --- a/src/qml/qml/v8/qv8contextwrapper.cpp +++ b/src/qml/qml/v8/qv8contextwrapper.cpp @@ -55,7 +55,7 @@ class QV8ContextResource : public QV8ObjectResource V8_RESOURCE_TYPE(ContextType); public: - QV8ContextResource(QV8Engine *engine, QQmlContextData *context, QObject *scopeObject); + QV8ContextResource(QV8Engine *engine, QQmlContextData *context, QObject *scopeObject, bool ownsContext = false); ~QV8ContextResource(); inline QQmlContextData *getContext() const; @@ -64,7 +64,8 @@ public: quint32 isSharedContext:1; quint32 hasSubContexts:1; quint32 readOnly:1; - quint32 dummy:29; + quint32 ownsContext:1; + quint32 dummy:28; // This is a pretty horrible hack, and an abuse of external strings. When we create a // sub-context (a context created by a Qt.include() in an external javascript file), @@ -86,15 +87,15 @@ private: }; -QV8ContextResource::QV8ContextResource(QV8Engine *engine, QQmlContextData *context, QObject *scopeObject) -: QV8ObjectResource(engine), isSharedContext(false), hasSubContexts(false), readOnly(true), - context(context), scopeObject(scopeObject) +QV8ContextResource::QV8ContextResource(QV8Engine *engine, QQmlContextData *context, QObject *scopeObject, bool ownsContext) +: QV8ObjectResource(engine), isSharedContext(false), hasSubContexts(false), readOnly(true), + ownsContext(ownsContext), context(context), scopeObject(scopeObject) { } QV8ContextResource::~QV8ContextResource() { - if (context && context->isJSContext) + if (context && ownsContext) context->destroy(); } @@ -186,7 +187,7 @@ v8::Local<v8::Object> QV8ContextWrapper::urlScope(const QUrl &url) // XXX NewInstance() should be optimized v8::Local<v8::Object> rv = m_urlConstructor->NewInstance(); - QV8ContextResource *r = new QV8ContextResource(m_engine, context, 0); + QV8ContextResource *r = new QV8ContextResource(m_engine, context, 0, true); rv->SetExternalResource(r); return rv; } @@ -226,6 +227,12 @@ QQmlContextData *QV8ContextWrapper::context(v8::Handle<v8::Value> value) return r?r->getContext():0; } +void QV8ContextWrapper::takeContextOwnership(v8::Handle<v8::Object> qmlglobal) +{ + QV8ContextResource *r = v8_resource_cast<QV8ContextResource>(qmlglobal); + r->ownsContext = true; +} + v8::Handle<v8::Value> QV8ContextWrapper::NullGetter(v8::Local<v8::String>, const v8::AccessorInfo &) { diff --git a/src/qml/qml/v8/qv8contextwrapper_p.h b/src/qml/qml/v8/qv8contextwrapper_p.h index 1e62ea6480..43eeee05d3 100644 --- a/src/qml/qml/v8/qv8contextwrapper_p.h +++ b/src/qml/qml/v8/qv8contextwrapper_p.h @@ -54,6 +54,7 @@ // #include <QtCore/qglobal.h> +#include <private/qtqmlglobal_p.h> #include <private/qv8_p.h> QT_BEGIN_NAMESPACE @@ -62,7 +63,7 @@ class QUrl; class QObject; class QV8Engine; class QQmlContextData; -class QV8ContextWrapper +class Q_QML_PRIVATE_EXPORT QV8ContextWrapper { public: QV8ContextWrapper(); @@ -84,6 +85,8 @@ public: inline v8::Handle<v8::Object> sharedContext() const; + void takeContextOwnership(v8::Handle<v8::Object> qmlglobal); + private: static v8::Handle<v8::Value> NullGetter(v8::Local<v8::String> property, const v8::AccessorInfo &info); |