From d67ce0fc99c2751733b8b1661095ce448b88cdbd Mon Sep 17 00:00:00 2001
From: "neil%parkwaycc.co.uk"
 <neil%parkwaycc.co.uk@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Fri, 4 Apr 2003 10:10:04 +0000
Subject: [PATCH] Fix JS warnings in previous checkin to bug 38367 r=varga
 sr=bz

git-svn-id: svn://10.0.0.236/trunk@140689 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../global/resources/content/bindings/popup.xml | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/mozilla/xpfe/global/resources/content/bindings/popup.xml b/mozilla/xpfe/global/resources/content/bindings/popup.xml
index 549ec045569..f899c5559a0 100644
--- a/mozilla/xpfe/global/resources/content/bindings/popup.xml
+++ b/mozilla/xpfe/global/resources/content/bindings/popup.xml
@@ -141,21 +141,20 @@
 
       <handler event="popupshowing" phase="target">
         <![CDATA[
-          // XXX Remove when phase="target" works
-          if (event.target != this)
-            return;
           var array = [];
           var width = 0;
           for (var menuitem = this.firstChild; menuitem; menuitem = menuitem.nextSibling) {
-            var accel = document.getAnonymousElementByAttribute(menuitem, "anonid", "accel");
-            if (accel && accel.label && accel.boxObject) {
-              array.push(accel);
-              if (accel.boxObject.width > width)
-                width = accel.boxObject.width;
+            if (menuitem.localName == "menuitem" && menuitem.hasAttribute("acceltext")) {
+              var accel = document.getAnonymousElementByAttribute(menuitem, "anonid", "accel");
+              if (accel && accel.boxObject) {
+                array.push(accel);
+                if (accel.boxObject.width > width)
+                  width = accel.boxObject.width;
+              }
             }
           }
           for (var i = 0; i < array.length; i++)
-            array[i].setAttribute("width", width);
+            array[i].width = width;
         ]]>
       </handler>
     </handlers>