Valgrind warning when theme is changed from MainViewStyle.qml

[Albert Astals Cid]

We are getting this warning in valgrind (with Qt 5.3)

==20334== Invalid read of size 8
==20334== at 0x581568A: QQmlObjectCreator::finalize(QQmlInstantiationInterrupt&) (qrecursionwatcher_p.h:94)
==20334== by 0x57A5A82: QQmlIncubatorPrivate::incubate(QQmlInstantiationInterrupt&) (qqmlincubator.cpp:357)
==20334== by 0x57A6215: QQmlEnginePrivate::incubate(QQmlIncubator&, QQmlContextData*) (qqmlincubator.cpp:95)
==20334== by 0x57A169C: QQmlComponent::create(QQmlIncubator&, QQmlContext*, QQmlContext*) (qqmlcomponent.cpp:1072)
==20334== by 0x5DCE449: QQuickLoaderPrivate::_q_sourceLoaded() (qquickloader.cpp:721)
==20334== by 0x5DCE5E7: QQuickLoaderPrivate::load() (qquickloader.cpp:604)
==20334== by 0x581564D: QQmlObjectCreator::finalize(QQmlInstantiationInterrupt&) (qqmlobjectcreator.cpp:1202)
==20334== by 0x57A1076: QQmlComponentPrivate::complete(QQmlEnginePrivate*, QQmlComponentPrivate::ConstructionState*) (qqmlcomponent.cpp:932)
==20334== by 0x57A1108: QQmlComponentPrivate::completeCreate() (qqmlcomponent.cpp:968)
==20334== by 0x57A0FCF: QQmlComponent::create(QQmlContext*) (qqmlcomponent.cpp:795)
==20334== by 0x5DD538E: QQuickView::continueExecute() (qquickview.cpp:489)
==20334== by 0x5DD5AB3: QQuickViewPrivate::execute() (qquickview.cpp:133)
==20334== by 0x407045: startShell(int, char const**, void*) (main.cpp:162)
==20334== by 0x407A74: main (main.cpp:227)
==20334== Address 0x20df8968 is 104 bytes inside a block of size 112 free'd
==20334== at 0x4C2C2BC: operator delete(void*) (vg_replace_malloc.c:503)
==20334== by 0x5815FB0: QQmlObjectCreator::~QQmlObjectCreator() (qqmlobjectcreator.cpp:156)
==20334== by 0x57A52AA: QQmlIncubatorPrivate::clear() (qscopedpointer.h:62)
==20334== by 0x57A53C6: QQmlIncubator::clear() (qqmlincubator.cpp:577)
==20334== by 0x5DCD7CF: QQuickLoaderPrivate::clear() (qquickloader.cpp:95)
==20334== by 0x5DCEB02: QQuickLoader::setSourceComponent(QQmlComponent*) (qquickloader.cpp:472)

Because MainViewStyle.qml is replacing the theme once it gets loaded (if it needs to go Dark) and we're basically changing the component in MainView.qml while the component is being loaded and it seems Qt doesn't like that.

Not sure if it could be argued this is a Qt 5.3 bug, haven't tried if the warning is there in Qt 5.2

[Albert Astals Cid (aacid)]

Just in case upstream actually may think it's an upstream bug i've reported https://bugreports.qt-project.org/browse/QTBUG-39775

[Albert Astals Cid (aacid)]

There's an upstream fix https://codereview.qt-project.org/#/c/88011/ i have not had time to check if it fixes it.

[Zsombor Egri (zsombi)]

Not checked with Valgrind, however the binding loops on properties were still there with the fix. The auto-theming got moved into MainView and is only applied for system themes.

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-ui-toolkit - 1.1.1364+15.04.20141209-0ubuntu1

---------------
ubuntu-ui-toolkit (1.1.1364+15.04.20141209-0ubuntu1) vivid; urgency=medium

  [ Zsombor Egri ]
  * Theming fixes: application theming, binding loop fix, auto-theming
    moved into MainView and it is applicable only on Ambiance and
    SuruDark, SuruGradient is no longer auto-themed.
    Fixes: LP: #1213043, LP: #1277647, LP: #1330510, LP: #1356779,
           LP: #1389792.
  * ServiceProperties component to Ubuntu.Components 1.1.
    Fixes: LP: #1391877.
  * New ListItem component base. Support for leading/trailing actions.
    Exported in Ubuntu.Components 1.2 UNSTABLE release.
    Fixes: LP: #1097728, LP: #1372592, LP: #1383870.
  * removing failing ServiceProperties and app-theming branches
  * Application theming ant other theming related binding loop fixes
  * ServiceProperties DBus property watcher

  [ Christian Dywan ]
  * Do the default theme check properly: for non-debug and without
    build errors.
  * Don't activate the control in ListItem.Standard implicitly.
    Fixes: LP: #1372563.
  * Improve AutoPilot caret dragging test cases.
  * Use new qt_gl_set_global_share_context setter with Qt 5.4.
    Fixes: LP: #1397979.
  * Add sortCaseSensitivity to the SortFilterModel example.
    Fixes: LP: #1391034.
  * Change offending links into bold font. Fixes: LP: #1395011.

  [ Daniel d'Andrada ]
  * This is needed by unity8 for the "shell rotation" feature (ie
    shell does all the rotation work instead of apps rotating
    themselves). unity8's dialogs (like the power dialog when you
    long press the power button) should follow its parent item
    rotation and size, as one would expect, and not be sneakily
    reparented to the scene root item, in which case the shell
    UI would be rotated but its dialog would not.
    Related bug LP: #1394511. Fixes: LP: #1394511.

  [ Tim Peeters ]
  * Remove the old header separator style. Fixes: LP: #1372590.

  [ Loïc Molinari ]
  * Made the UCAlarmPrivate destructor private.

  [ Ubuntu daily release ]
  * New rebuild forced
 -- Ubuntu daily release <email address hidden> Tue, 09 Dec 2014 11:47:33 +0000