1380999 - Bookmark toolbar is very slow to render if there are many bookmarks

Status: RESOLVED WORKSFORME

(Firefox :: Bookmarks & History, defect, P3)

Description

[Kacper Michajłow [:kasper93]]

Hi,

I have over 1000 bookmarks on the bookmark toolbar. Don't as me why. The problem is that initialization of that toolbar takes few seconds (5-6) and I have quite decent CPU so the issue might get more significant on lower end machines. Opening new window is painful. With bookmark toolbar disabled new window loads almost instantly.

It looks like Firefox is rendering all the elements which takes a long time. I think there is opportunity to improve performance there. Either by implementing lazy loading, I mean there are only dozen elements on toolbar, the rest are not visible. Or just try to optimize whatever is happening with the bookmarks to improve window loading time.

I know my example is extreme, but I think that improving performance in this area is critical, because it translated directly to Firefox window open time. So even if people dosn't have so many bookmarks on the toolbar, still they shouldn't take so much time to load.

Regards,
Kacper

Comment 1

[Tooru Fujisawa [:arai]]

do you mean 1000 items directly under bookmark toolbar, or including items inside folders under bookmark toolbar?

Comment 2

[Kacper Michajłow [:kasper93]]

I mean items directly under bookmark toolbar. The exact number in irrelevant. The point here is that "a lot" of items causes significant lag during window initialization. While only a few of bookmarks items are visible. I think there is opportunity for improvements here.

Comment 3

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

If you're a nightly user, please test the next nightly including bug 1392081 and let me know.

Comment 4

[Kacper Michajłow [:kasper93]]

I see noticeable improvement. Now new windows are opened very quickly. Great work!

Sorry, for late reply, I was away for two weeks and couldn't test it.

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Closing as works-for-me based on comment 4.