Bug 405235: land the simpler fix for broken search suggest: unhide our panel explicitly since we're overriding the openPopup method that would have unhidden it otherwise, patch by Seth Spitzer <sspitzer@mozilla.com>, r=me/dao

2007-12-04 22:54:34 +00:00 · 2007-12-04 22:54:34 +00:00 · 2e8257f137
--- a/browser/components/search/content/search.xml
+++ b/browser/components/search/content/search.xml
@ -649,6 +649,14 @@
        <body><![CDATA[
          var popup = this.popup;
          if (!popup.mPopupOpen) {
+            // Initially the panel used for the searchbar (PopupAutoComplete
+            // in browser.xul) is hidden to avoid impacting startup / new
+            // window performance. The base binding's openPopup would normally
+            // call the overriden openAutocompletePopup in urlbarBindings.xml's
+            // urlbar-result-popup binding to unhide the popup, but since we're
+            // overriding openPopup we need to unhide the panel ourselves.
+            popup.hidden = false;
+
            popup.mInput = this;
            popup.view = this.controller.QueryInterface(Components.interfaces.nsITreeView);
            popup.invalidate();