In bug 1312999 I added code to cache the one-off buttons and rebuild them only if the textbox size changed. Unfortunately, the current caller code sets the popup before the textbox. Setting popup causes a _rebuild call, which can't cache the textbox size as the textbox setter hasn't been called yet. This causes the reflows in bug 1357054 to happen both the first and second time the awesomebar or searchbar panel is opened, instead of only the first time.

This wasn't a problem before bug 1369705 as both the popup and textbox setters were called before we had any panel open.

Comment on attachment 8889459 [details] [diff] [review] Patch Review of attachment 8889459 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/base/content/urlbarBindings.xml @@ +1684,5 @@ > this._oneOffSearchesEnabled = enable; > if (enable) { > this.oneOffSearchButtons.telemetryOrigin = "urlbar"; > this.oneOffSearchButtons.style.display = "-moz-box"; > + // The popup setter will cause a _rebuild call, which uses .textbox Maybe I'd rephrase both comments as // Set .textbox first, since the popup setter will cause // a _rebuild call that uses it. otherwise it's a bit unclear what the comment tries to underline.

Pushed by florian@queze.net: https://hg.mozilla.org/integration/mozilla-inbound/rev/91a2c24475fa One-off buttons should be cached the first time, r=mak.