Bug 578440 Use <menupopup> instead of <popup>. r=Neil a=Callek (over IRC) CLOSED TREE.

suite/browser/navigatorOverlay.xul

@@ -224,7 +224,7 @@
          
   <!-- bookmarks context menu -->
   <popupset id="bookmarksPopupset">
-    <popup id="bookmarks-context-menu" 
+    <menupopup id="bookmarks-context-menu" 
            onpopupshowing="return BookmarksMenu.createContextMenu(event);"
            onpopuphidden ="BookmarksMenu.destroyContextMenu(event);"/>
   </popupset>

suite/common/downloads/downloadmanager.xul

@@ -147,7 +147,7 @@
   </keyset>
 
   <popupset id="downloadPopupset">
-    <popup id="downloadContext">
+    <menupopup id="downloadContext">
       <menuitem id="dlContext-pause"
                 label="&cmd.pause.label;"
                 accesskey="&cmd.pause.accesskey;"
@@ -192,7 +192,7 @@
                 command="cmd_properties"/>
       <menuseparator/>
       <menuitem id="cMenu_selectAll"/>
-    </popup>
+    </menupopup>
   </popupset>
 
   <toolbox id="download-toolbox">

suite/common/history/history-panel.xul

@@ -66,7 +66,7 @@
     <key id="key_delete2"/>
   </keyset>
 
-  <popup id="placesContext"/>
+  <menupopup id="placesContext"/>
 
   <hbox align="center">
     <textbox id="search-box" flex="1" type="search"

suite/common/history/history.xul

@@ -102,7 +102,7 @@
   </keyset>
 
   <popupset id="placesPopupset">
-    <popup id="placesContext"/>
+    <menupopup id="placesContext"/>
   </popupset>
 
   <toolbox id="history-toolbox">

suite/common/history/placesOverlay.xul

@@ -82,7 +82,7 @@
              oncommand="goDoCommand('placesCmd_delete:domain');"/>
   </commandset>
 
-  <popup id="placesContext"
+  <menupopup id="placesContext"
          onpopupshowing="this._view = PlacesUIUtils.getViewForNode(document.popupNode);
                          return this._view.buildContextMenu(this);"
          onpopuphiding="this._view.destroyContextMenu();">
@@ -138,7 +138,7 @@
               label="&selectAllCmd.label;"
               accesskey="&selectAllCmd.accesskey;"
               selection="any"/>
-  </popup>
+  </menupopup>
 
   <menupopup id="viewPopup">
     <menu id="viewColumns"

suite/common/search/internetresults.xul

@@ -12,7 +12,7 @@
       context="disallowcontext">
 
   <popupset>
-    <popup id="disallowcontext" onpopupshowing="return false;"/>
+    <menupopup id="disallowcontext" onpopupshowing="return false;"/>
   </popupset>
 
   <script type="application/javascript" src="chrome://global/content/nsUserSettings.js"/>

suite/common/search/search-panel.xul

@@ -57,9 +57,9 @@
   <stringbundle id="regionalBundle" src="chrome://communicator-region/locale/region.properties"/>
   
   <popupset>
-    <popup id="contextual" onpopupshowing="return fillContextMenu('contextual', 'resultList');" >
-      <menu />
-    </popup>
+    <menupopup id="contextual" onpopupshowing="return fillContextMenu('contextual', 'resultList');">
+      <menu/>
+    </menupopup>
   </popupset>
 
   <popupset id="descTooltipSet">

suite/mailnews/addrbook/addressbook.xul

@@ -162,7 +162,7 @@
 
 </keyset>
 
-<popup id="dirTreeContext">
+<menupopup id="dirTreeContext">
   <menuitem id="dirTreeContext-properties"
             label="&editItemButton.label;"
             accesskey="&editItemButton.accesskey;"
@@ -181,9 +181,9 @@
             label="&deleteItemButton.label;"
             accesskey="&deleteItemButton.accesskey;"
             command="button_delete"/>
-</popup>
+</menupopup>
 
-<popup id="abResultsTreeContext">
+<menupopup id="abResultsTreeContext">
   <menuitem id="abResultsTreeContext-properties"
             label="&editItemButton.label;"
             accesskey="&editItemButton.accesskey;"
@@ -202,7 +202,7 @@
             label="&deleteItemButton.label;"
             accesskey="&deleteItemButton.accesskey;"
             command="button_delete"/>
-</popup>
+</menupopup>
 
 <toolbox class="toolbox-top">
     <menubar id="ab-menubar" grippytooltiptext="&menuBar.tooltip;">