Bug 223277: real fix concerning the event chain + Band aid patch for an XBL bug.

browser/components/search/content/search.xml

@@ -146,20 +146,22 @@
 
     <handlers>
 
-      <handler event="keypress" keycode="vk_up" modifiers="control">
+      <handler event="keypress" keycode="vk_up" modifiers="control" phase="capturing">
         <![CDATA[
           var currentEngine = this.currentEngine;
-          if (!currentEngine || currentEngine == "__PhoenixFindInPage")
+          if (currentEngine && currentEngine != "__PhoenixFindInPage") {
-            return;
+            var searchEngineMenuitem = document.getElementById(currentEngine);
-          var searchEngineMenuitem = document.getElementById(currentEngine);
+            var newEngine = searchEngineMenuitem.previousSibling;
-          var newEngine = searchEngineMenuitem.previousSibling;
+            if (!newEngine.id)
-          if (!newEngine.id)
+              newEngine = {id: "__PhoenixFindInPage"}
-            newEngine = {id: "__PhoenixFindInPage"}
+            this.currentEngine = newEngine.id;
-          this.currentEngine = newEngine.id;
+          }
+          event.preventDefault();
+          event.stopPropagation();
         ]]>
       </handler>
 
-      <handler event="keypress" keycode="vk_down" modifiers="control">
+      <handler event="keypress" keycode="vk_down" modifiers="control" phase="capturing">
         <![CDATA[
           var currentEngine = this.currentEngine;
           if (!currentEngine)
@@ -173,6 +175,8 @@
           }
           if (newEngine)
             this.currentEngine = newEngine.id;
+          event.preventDefault();
+          event.stopPropagation();
         ]]>
       </handler>