Bug 1335992 - Search with default search engine stops working. r=florian

MozReview-Commit-ID: 9GnA9Y3YvTS

--HG--
extra : rebase_source : f44b3fff39b18ac2d806346b64a5c454d1cd7d96

browser/components/search/content/search.xml

@@ -1114,6 +1114,21 @@
           this._isHiding = false;
         }, 0);
       ]]></handler>
+
+      <!-- This handles clicks on the topmost "Foo Search" header in the
+           popup (hbox[anonid="searchbar-engine"]). -->
+      <handler event="click"><![CDATA[
+        if (event.button == 2) {
+          // Ignore right clicks.
+          return;
+        }
+        let button = event.originalTarget;
+        let engine = button.parentNode.engine;
+        if (!engine) {
+          return;
+        }
+        this.oneOffButtons.handleSearchCommand(event, engine);
+      ]]></handler>
     </handlers>
 
   </binding>
@@ -2223,7 +2238,7 @@
           return; // ignore right clicks.
 
         let button = event.originalTarget;
-        let engine = button.engine || button.parentNode.engine;
+        let engine = button.engine;
 
         if (!engine)
           return;

browser/components/search/test/browser_oneOffHeader.js

@@ -122,10 +122,19 @@ add_task(function* test_text() {
   is(getHeaderText(), "Search for foo with:",
      "Header has the correct text when search terms have been entered and the Change Search Settings button is selected.");
 
-  promise = promiseEvent(searchPopup, "popuphidden");
+  // Click the "Foo Search" header at the top of the popup and make sure it
-  info("Closing search panel");
+  // loads the search results.
-  EventUtils.synthesizeKey("VK_ESCAPE", {});
+  let searchbarEngine =
-  yield promise;
+    document.getAnonymousElementByAttribute(searchPopup, "anonid",
+                                            "searchbar-engine");
+
+  yield synthesizeNativeMouseMove(searchbarEngine);
+  SimpleTest.executeSoon(() => {
+    EventUtils.synthesizeMouseAtCenter(searchbarEngine, {});
+  });
+
+  let url = Services.search.currentEngine.getSubmission(textbox.value).uri.spec;
+  yield promiseTabLoadEvent(gBrowser.selectedTab, url);
 
   // Move the cursor out of the panel area to avoid messing with other tests.
   yield synthesizeNativeMouseMove(searchbar);