Improve the performance of the new tree list box
Categories
(Thunderbird :: Folder and Message Lists, defect)
Tracking
(thunderbird_esr102 wontfix)
| Tracking | Status | |
|---|---|---|
| thunderbird_esr102 | --- | wontfix |
People
(Reporter: aleca, Assigned: darktrojan)
References
(Blocks 1 open bug)
Details
(Keywords: perf)
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
Scrolling through a folder with 100k+ messages seems to cause RAM consumption to go up (13GB) and CPU spikes.
|
||
Updated•2 years ago
|
|
Assignee | |
Comment 1•2 years ago
|
||
This code didn't take into account the fact that the new scroll position could be completely outside the current position, so it creates every row in between, only to remove them again.
|
||
Updated•2 years ago
|
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/f227dd8cd534
When scrolling the thread tree, don't create rows that are never seen. r=aleca
|
|
Assignee | |
Updated•2 years ago
|
|
|
||
Updated•2 years ago
|
|
||
Comment 3•2 years ago
|
||
Hey guys, both Alex and Geoff: thanks so much for noticing and fixing this bug. I see in the patch that it was a simple bug, and that the design was sound, so congratulations both for the design and for taking care.
|
Reporter | |
Comment 4•2 years ago
|
||
(In reply to Ben Bucksch (:BenB) from comment #3)
Hey guys, both Alex and Geoff: thanks so much for noticing and fixing this bug. I see in the patch that it was a simple bug, and that the design was sound, so congratulations both for the design and for taking care.
Thank you so much for this message!
There are still a lot of performance issues that we identified and can improve with very simple changes.
It will get better and better!
Description
•