Regression: Performance hit when editing sorted lists
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu Notes app |
Fix Released
|
Critical
|
Unassigned | ||
qtdeclarative-opensource-src (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
This is a follow-up from bug 1303746.
After the migration from Qt 5.2.1 to 5.3.0 in utopic, it seems that this patch was lost, and it did not land upstream until 5.3.1, which we do not ship yet. Therefore, we've regressed and we're back to the same bug.
The most visible effect is in the Reminders app, which uses a list sorted by date to show the existing notes, and that list is continually being reordered, rendering the application unusable.
[Impact]
Performance regression from Qt 5.2.1 to 5.3.0 in sorted lists, due to lacking a proper way to support layoutChange.
[Test Case]
We first noticed this after the migration to 5.3.0 in utopic and using the Reminders app: in summary, when editing a note the application becomes unresponsive on the phone and the CPU goes to 100% on the desktop (there it's not too noticeable).
From the upstream bug, my understanding is that the fix landed after Qt 5.2.1 was released and it made it to 5.3.1, but not 5.3.0, which is what we currently ship in Ubuntu.
Related branches
- PS Jenkins bot: Approve (continuous-integration)
- Kubuntu Packagers: Pending requested
-
Diff: 208 lines (+169/-1)4 files modifieddebian/changelog (+8/-0)
debian/libqt5qml5.symbols (+2/-1)
debian/patches/Implement-proper-support-for-layoutChange-in-QQmlDel.patch (+158/-0)
debian/patches/series (+1/-0)
Changed in reminders-app: | |
status: | New → Triaged |
importance: | Undecided → Critical |
summary: |
- Regressioin: Performance hit when editing sorted lists + Regression: Performance hit when editing sorted lists |
Changed in qtdeclarative-opensource-src (Ubuntu): | |
status: | New → Confirmed |
I can confirm that this upload fixes the bug. Tested on image 100 and ran the Unity test plan - the manual part, but not the autopilot tests.