Comment 0 for bug 1505731

Revision history for this message
Ugo Riboni (uriboni) wrote :

In the mobile version of the new tab view, when the "more" button is activated, a view of all the bookmark folders is revealed.
Each bookmark folder is implemented as a Column with a Repeater inside, creating one Loader item for each bookmark in the folder.
Even if most of the Loaders remain inactive they still have a memory and initialization time cost, therefore the memory usage and initialization time of this view raises proportionally to the number of bookmarks in the database.

If the entire view could use instead a single ListView with one section per bookmark, then these costs would be avoided since the ListView maintains only delegates for the items that are visible at the moment.

This would likely require empty folders to not be hidden, since the ListView can only create sections when there is at least one item in that section.