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
|
Timo Jyrinki | ||
Trusty |
Fix Released
|
Undecided
|
Timo Jyrinki |
Bug Description
[Impact]
Performance regression from saucy in sorted lists, due to lacking a proper way to support layoutChange.
[Test Case]
We first noticed this after the migration to Qt 5.2 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).
[Regression Potential]
No regressions spotted in utopic with the same patch. List views could be the place where also potential regression would be seen most likely.
---
We first noticed this after the migration to Qt 5.2 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).
After a chat with mzanetti, this is related to an upstream patch that should have probably been implemented differently, and it's now reported in the upstream tracker [1].
This will affect every app using a sorted list on the phone
Related branches
- PS Jenkins bot: Approve (continuous-integration)
- Kubuntu Packagers: Pending requested
-
Diff: 931 lines (+684/-29)13 files modifieddebian/changelog (+50/-0)
debian/control (+156/-26)
debian/copyright (+20/-0)
debian/libqt5qml5.symbols (+2/-1)
debian/mark_private_symbols.sh (+20/-0)
debian/patches/Fix-marking-of-prototype-objects-in-chain.patch (+89/-0)
debian/patches/Implement-proper-support-for-layoutChange-in-QQmlDel.patch (+158/-0)
debian/patches/QQuickItemView-QQuickPathView-Fix-creation-of-delega.patch (+128/-0)
debian/patches/V4-regalloc-fix-register-spill-choice-under-high-pre.patch (+50/-0)
debian/patches/series (+4/-0)
debian/qml-module-qtquick-dialogs.install (+5/-1)
debian/qml-module-qtquick-dialogs.lintian-overrides (+1/-1)
debian/source/lintian-overrides (+1/-0)
Changed in reminders-app: | |
status: | New → Triaged |
importance: | Undecided → Critical |
Changed in qtdeclarative-opensource-src (Ubuntu): | |
assignee: | nobody → Zsombor Egri (zsombi) |
Changed in qtdeclarative-opensource-src (Ubuntu): | |
assignee: | Zsombor Egri (zsombi) → Timo Jyrinki (timo-jyrinki) |
Changed in qtdeclarative-opensource-src (Ubuntu Trusty): | |
assignee: | nobody → Timo Jyrinki (timo-jyrinki) |
status: | New → In Progress |
description: | updated |
description: | updated |
Changed in reminders-app: | |
status: | Triaged → Fix Committed |
Changed in reminders-app: | |
status: | Fix Committed → Fix Released |
Status changed to 'Confirmed' because the bug affects multiple users.