From 222611e4a61b8c2dd627d94115790fe41cae6e93 Mon Sep 17 00:00:00 2001
From: "blakeross%telocity.com" <blakeross%telocity.com>
Date: Thu, 10 Oct 2002 01:47:29 +0000
Subject: [PATCH] Fix checkbox for block images.

---
 browser/base/content/browser.xul | 6 ++++--
 toolkit/content/nsContextMenu.js | 6 +++++-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/browser/base/content/browser.xul b/browser/base/content/browser.xul
index 23b19d9460c..0b69e76cabe 100644
--- a/browser/base/content/browser.xul
+++ b/browser/base/content/browser.xul
@@ -321,9 +321,11 @@ Contributor(s):
                 label="&setWallpaperCmd.label;"
                 accesskey="&setWallpaperCmd.accesskey;"
                 oncommand="gContextMenu.setWallpaper();"/>
+
+<!--XXXBlake XUL sucks ass! We need autocheck for menuitems. -->
       <menuitem id="context-blockimage"
-                label="&blockImageCmd.label;" type="checkbox"            
-                oncommand="gContextMenu.toggleImageBlocking(!this.checked);"/>
+                label="&blockImageCmd.label;" class="menuitem-iconic"
+                oncommand="gContextMenu.toggleImageBlocking(this.getAttribute('checked') != 'true');"/>
       <menuitem id="context-back"
                 label="&backCmd.label;"
                 accesskey="&backCmd.accesskey;"
diff --git a/toolkit/content/nsContextMenu.js b/toolkit/content/nsContextMenu.js
index 2720237dd21..c67ad57d2c6 100644
--- a/toolkit/content/nsContextMenu.js
+++ b/toolkit/content/nsContextMenu.js
@@ -151,7 +151,11 @@ nsContextMenu.prototype = {
         this.showItem( "context-blockimage", this.onImage);
         if (this.onImage) {
           var blockImage = document.getElementById("context-blockimage");
-          blockImage.checked = this.isImageBlocked();
+          if (this.isImageBlocked()) {
+            blockImage.setAttribute("checked", "true");
+          }
+          else
+            blockImage.removeAttribute("checked");
         }
     },
     initClipboardItems : function () {