From b0ea346ec9513995c0860ef9133e9ff00a11eb52 Mon Sep 17 00:00:00 2001
From: "ben%netscape.com" <ben%netscape.com>
Date: Tue, 6 Nov 2001 01:39:36 +0000
Subject: [PATCH] 108595 - "Block Images for this server" menu item not showing
 up. Remove display:none style rule now that the overlay that includes this
 overlay (contentAreaContextOverlay.xul) is no longer being loaded twice by
 navigator. r=kerz, sr=blake

---
 .../cookie/resources/content/cookieContextOverlay.xul     | 8 --------
 .../permissions/content/imageContextOverlay.xul           | 8 --------
 2 files changed, 16 deletions(-)

diff --git a/extensions/cookie/resources/content/cookieContextOverlay.xul b/extensions/cookie/resources/content/cookieContextOverlay.xul
index 0f60c23972e9..e3399cb60edc 100644
--- a/extensions/cookie/resources/content/cookieContextOverlay.xul
+++ b/extensions/cookie/resources/content/cookieContextOverlay.xul
@@ -127,12 +127,6 @@
 
     window.addEventListener("load", cookieContextMenu.addContextMenuItemListeners, false);
 
-    // For some unexplainable reason, this overlay is loaded twice as can be demonstrated
-    // by uncommenting the following "dump" statement which will get displayed twice
-    //    dump("$$$$$$$$$$ HERE WE ARE IN cookieContextOverlay.xul $$$$$$$$$$\n");
-    // As a consequence, the block-image item appears twice in the context menu.  To
-    // prevent that from happening, the "display:none" was added to the menuitem below
-
    ]]>
   </script>         
 
@@ -142,13 +136,11 @@
               label="&blockImageCmd.label;"
               accesskey=""
               oncommand="cookieContextMenu.blockImage();"
-              style="display:none;"
               insertafter="context-viewimage"/>
     <menuitem id="context-unblockimage"
               label="&unblockImageCmd.label;"
               accesskey=""
               oncommand="cookieContextMenu.unblockImage();"
-              style="display:none;"
               insertafter="context-viewimage"/>
   </popup>
 </overlay>
diff --git a/xpfe/components/permissions/content/imageContextOverlay.xul b/xpfe/components/permissions/content/imageContextOverlay.xul
index 0f60c23972e9..e3399cb60edc 100644
--- a/xpfe/components/permissions/content/imageContextOverlay.xul
+++ b/xpfe/components/permissions/content/imageContextOverlay.xul
@@ -127,12 +127,6 @@
 
     window.addEventListener("load", cookieContextMenu.addContextMenuItemListeners, false);
 
-    // For some unexplainable reason, this overlay is loaded twice as can be demonstrated
-    // by uncommenting the following "dump" statement which will get displayed twice
-    //    dump("$$$$$$$$$$ HERE WE ARE IN cookieContextOverlay.xul $$$$$$$$$$\n");
-    // As a consequence, the block-image item appears twice in the context menu.  To
-    // prevent that from happening, the "display:none" was added to the menuitem below
-
    ]]>
   </script>         
 
@@ -142,13 +136,11 @@
               label="&blockImageCmd.label;"
               accesskey=""
               oncommand="cookieContextMenu.blockImage();"
-              style="display:none;"
               insertafter="context-viewimage"/>
     <menuitem id="context-unblockimage"
               label="&unblockImageCmd.label;"
               accesskey=""
               oncommand="cookieContextMenu.unblockImage();"
-              style="display:none;"
               insertafter="context-viewimage"/>
   </popup>
 </overlay>