Delete JS-owned QML objects right away in the engine dtor.

This prevents memory leaks when the engine is destroyed after exec()
has already finished. In most cases this happens during application
shutdown, at which point the event loop is never entered again.

Task-number: QTBUG-20377
Change-Id: I65564ed3e56314d656d92fd66f11ae67d4eb932b
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@nokia.com>
Reviewed-by: Lars Knoll <lars.knoll@nokia.com>

2 files changed, 13 insertions, 1 deletions

diff --git a/src/script/bridge/qscriptdeclarativeclass_p.h b/src/script/bridge/qscriptdeclarativeclass_p.h
index c8c35bf..c67d23d 100644
--- a/src/script/bridge/qscriptdeclarativeclass_p.h
+++ b/src/script/bridge/qscriptdeclarativeclass_p.h
@@ -77,7 +77,14 @@ public:
 
     typedef void* Identifier;
 
-    struct Object { virtual ~Object() {} };
+    struct Object {
+        // Like the destructor, frees resources associated with this object. The difference is that
+        // the destructor might delay resource freeing with deleteLater(), while disposeNow() always
+        // deletes the resource straigt away.
+        virtual void disposeNow() {}
+
+        virtual ~Object() {}
+    };
 
     static QScriptValue newObject(QScriptEngine *, QScriptDeclarativeClass *, Object *);
     static Value newObjectValue(QScriptEngine *, QScriptDeclarativeClass *, Object *);
diff --git a/src/script/bridge/qscriptdeclarativeobject.cpp b/src/script/bridge/qscriptdeclarativeobject.cpp
index 927309a..a9cd87d 100644
--- a/src/script/bridge/qscriptdeclarativeobject.cpp
+++ b/src/script/bridge/qscriptdeclarativeobject.cpp
@@ -53,6 +53,11 @@ DeclarativeObjectDelegate::DeclarativeObjectDelegate(QScriptDeclarativeClass *c,
 
 DeclarativeObjectDelegate::~DeclarativeObjectDelegate()
 {
+    // When the engine is being destructed, delete the object now, instead of using deleteLater(),
+    // to not have memory leaks on exit.
+    if (m_class->engine() && QScriptEnginePrivate::get(m_class->engine())->inDestructor)
+        m_object->disposeNow();
+
     delete m_object;
 }