1019991 - about:newtab should lazily build its search panel popup

Status: RESOLVED FIXED

(Firefox :: New Tab Page, defect)

Description

[Drew Willcoxon :adw]

Right now, about:newtab populates its search panel popup as soon as it receives the current search state from ContentSearch.jsm.  Instead, it should populate the panel when the panel is opened.

This may help fix some of the TART regression in bug 1004429, although I wasn't able to prove it using similar patches on tryserver and the various talos tools.  It trades work done on visibility change (when the page moves out of the preloader) for work done when you open the panel, but I think that's actually a good trade-off since it's less likely to regress TART, the amount of work is small and unlikely to cause a noticable delay when you open the panel, and if you never open the panel -- and right now, it's hard to open the panel because many providers don't have logos -- you don't pay for the work.

Comment 1

[Drew Willcoxon :adw]

Attachment: patch

This builds on patch 2 in bug 1019989.

https://tbpl.mozilla.org/?tree=Try&rev=fb32d078180d

Comment 2

[Tim Taubert [:ttaubert] (inactive)]

Comment on attachment 8440248 [details] [diff] [review]
patch

Review of attachment 8440248 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM!

Comment 3

[Drew Willcoxon :adw]

https://hg.mozilla.org/integration/fx-team/rev/79f665a0e949

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/79f665a0e949