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.

This builds on patch 2 in bug 1019989. https://tbpl.mozilla.org/?tree=Try&rev=fb32d078180d

Comment on attachment 8440248 [details] [diff] [review] patch Review of attachment 8440248 [details] [diff] [review]: ----------------------------------------------------------------- LGTM!