Activity log for bug #1552811

Date Who What changed Old value New value Message
2016-03-03 17:11:18 Gustavo Pichorim Boiko bug added bug
2016-03-03 17:11:45 Gustavo Pichorim Boiko messaging-app (Ubuntu): importance Undecided Medium
2016-03-03 17:11:46 Gustavo Pichorim Boiko messaging-app (Ubuntu): assignee Gustavo Pichorim Boiko (boiko)
2016-03-03 17:12:12 Gustavo Pichorim Boiko branch linked lp:~boiko/messaging-app/fix_thread_removing
2016-03-04 15:55:28 Bill Filler bug task added canonical-devices-system-image
2016-03-04 15:56:56 Bill Filler canonical-devices-system-image: assignee Bill Filler (bfiller)
2016-03-04 15:56:59 Bill Filler canonical-devices-system-image: importance Undecided Medium
2016-03-08 21:25:32 Pat McGowan description Depending on the contents of the page, its asynchronous creation might take a really long time because of the internal behavior (or bug?) of QtQuick. Loading pages synchronously fixes the problem, but the way it is implemented in messaging-app, there is a risk of leaking memory due to pages not being deleted properly when switching views. There is another way to workaround this QtQuick limitation which is to force the page creation to finish before returning to the event loop, in the AdaptivePageLayout context, it would look like this: var incubator = layout.addPageToNextColumn(originalPage, newPage, properties) incubator.forceCompletion() In order for this to be fixed, there is one but in AdaptivePageLayout which needs to be fixed (#1544745). Depending on the contents of the page, its asynchronous creation might take a really long time because of the internal behavior (or bug?) of QtQuick. Loading pages synchronously fixes the problem, but the way it is implemented in messaging-app, there is a risk of leaking memory due to pages not being deleted properly when switching views. There is another way to workaround this QtQuick limitation which is to force the page creation to finish before returning to the event loop, in the AdaptivePageLayout context, it would look like this: var incubator = layout.addPageToNextColumn(originalPage, newPage, properties) incubator.forceCompletion() In order for this to be fixed bug #1544745 in AdaptivePageLayout first needs to be fixed.
2016-03-08 21:25:51 Pat McGowan canonical-devices-system-image: status New In Progress
2016-03-08 21:25:55 Pat McGowan messaging-app (Ubuntu): status New In Progress