Merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-lok-zoom-leak into lp:ubuntu-docviewer-app

[Stefano Verzegnassi]

[loview] Fixed leak and tiles that stay visible even after their removal

* [loview] Fixed memory leak
* [loview] Fixed tiles that stay visible even after their removal

I forgot to delete the allocated memory when we remove a tile from the QMap during a zoom event.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:187
http://91.189.93.70:8080/job/ubuntu-docviewer-app-reboot-ci/43/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-docviewer-app-reboot-vivid-amd64-ci/43

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-docviewer-app-reboot-ci/43/rebuild

review: Approve (continuous-integration)

[Roman Shchekin (mrqtros)]

Great!

review: Approve

=== modified file 'src/plugin/libreofficetoolkit-qml-plugin/loview.cpp'
--- src/plugin/libreofficetoolkit-qml-plugin/loview.cpp	2015-10-11 11:27:29 +0000
+++ src/plugin/libreofficetoolkit-qml-plugin/loview.cpp	2015-10-12 10:54:43 +0000
@@ -370,7 +370,12 @@
     for (auto i = m_tiles.begin(); i != m_tiles.end(); ++i)
         RenderEngine::instance()->dequeueTask(i.value()->id());
 
-    m_tiles.clear();
+    auto i = m_tiles.begin();
+    while (i != m_tiles.end()) {
+        SGTileItem* sgtile = i.value();
+        sgtile->deleteLater();
+        i = m_tiles.erase(i);
+    }
 }
 
 LOView::~LOView()

=== modified file 'src/plugin/libreofficetoolkit-qml-plugin/sgtileitem.cpp'
--- src/plugin/libreofficetoolkit-qml-plugin/sgtileitem.cpp	2015-10-11 11:27:29 +0000
+++ src/plugin/libreofficetoolkit-qml-plugin/sgtileitem.cpp	2015-10-12 10:54:43 +0000
@@ -28,13 +28,15 @@
     QQuickWindow* wnd = window();
 
     if (!node && wnd && !m_data.isNull()) {
-        QImage image = m_data;
-        auto texture = wnd->createTextureFromImage(image);
+        auto texture = wnd->createTextureFromImage(m_data);
         node = new QSGSimpleTextureNode();
         node->setTexture(texture);
         node->setOwnsTexture(true);
         node->setRect(m_area);
 
+        // We don't need anymore QImage's data
+        m_data = QImage();
+
 #ifdef DEBUG_SHOW_TILE_BORDER
         drawTileBorders(node);
 #endif