1383749 - One-off buttons should be cached the first time

Status: RESOLVED FIXED

(Firefox :: Search, defect, P1)

Description

[Florian Quèze [:florian]]

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.

Comment 1

[Florian Quèze [:florian]]

Attachment: Patch

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

Comment 2

[Marco Bonardo [:mak]]

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.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/91a2c24475fa
One-off buttons should be cached the first time, r=mak.

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/91a2c24475fa