Fixing 70746: major xul syntax changes. If you use xul, this WILL affect you! Read n.p.m.xpfe for details on the changes and how you can fix your code. r=hewitt sr=hyatt, ben XPCDOM_20010329_BASE
authorblakeross%telocity.com
Wed, 21 Mar 2001 19:57:29 -0500
changeset 16 f6278eaf61c76cca2fbf6bb51e219e53b8b329de
parent 15 99ceca68d81c00534d1973b28ab1777808f4665a
child 17 ca62530369f98cb54810c0e2a6af3639d21c00e2
push id1
push usersdwilsh@shawnwilsher.com
push dateTue, 08 Jul 2008 16:35:39 +0000
reviewershewitt, hyatt, ben
bugs70746
Fixing 70746: major xul syntax changes. If you use xul, this WILL affect you! Read n.p.m.xpfe for details on the changes and how you can fix your code. r=hewitt sr=hyatt, ben
resources/content/ViewerPane.js
resources/content/extensions/treeEditable.xml
resources/content/jsutil/xul/inFormManager.js
resources/content/jsutil/xul/inTreeTableBuilder.js
resources/content/popupOverlay.xul
resources/content/prefs/pref-inspector.xul
resources/content/prefs/pref-sidebar.xul
resources/content/prefs/prefsOverlay.xul
resources/content/search/modules/findFiles/dialog.xul
resources/content/search/modules/junkImgs/dialog.xul
resources/content/search/modules/junkImgs/junkImgs.xml
resources/content/search/modules/popupOverlay.xul
resources/content/sidebar/sidebar.xul
resources/content/tasksOverlay.xul
resources/content/toolboxOverlay.xul
resources/content/viewers/dom/dom.xul
resources/content/viewers/dom/findDialog.xul
resources/content/viewers/dom/popupOverlay.xul
resources/content/viewers/nodeElement/nodeElement.xul
resources/content/viewers/styleRules/styleRules.xul
--- a/resources/content/ViewerPane.js
+++ b/resources/content/ViewerPane.js
@@ -1,29 +1,29 @@
 /***************************************************************
 * ViewerPane ---------------------------------------------------
 *  Interface for a pane accepts a node and displays all eligible
 *  viewers in a list and activates the selected viewer.
-* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
+* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 * REQUIRED IMPORTS:
 *   chrome://inspector/content/jsutil/xpcom/XPCU.js
 ****************************************************************/
 
 //////////// global variables /////////////////////
 
 //////////// global constants ////////////////////
 
 ////////////////////////////////////////////////////////////////////////////
 //// class ViewerPane
 
 function ViewerPane() // implements inIViewerPane
 {
 }
 
-ViewerPane.prototype = 
+ViewerPane.prototype =
 {
   ////////////////////////////////////////////////////////////////////////////
   //// Initialization
 
   mContextMenu: null,
   mCurrentEntry: null,
   mCurrentViewer: null,
 
@@ -35,17 +35,17 @@ ViewerPane.prototype =
   get viewee() { return this.mViewee; },
   set viewee(aVal) { this.startViewingObject(aVal) },
 
   get container() { return this.mContainer; },
   get viewer() { return this.mCurrentViewer; },
   get title() { return this.mTitle; },
   get uiElement() { return this.mUIElement; },
   get viewerReg() { return this.mViewerReg; },
-  
+
   //// methods
 
   initialize: function (aTitle, aContainer, aUIElement, aRegistry)
   {
     this.mTitle = aTitle;
     this.mUIElement = aUIElement;
     this.mContainer = aContainer;
     this.mViewerReg = aRegistry;
@@ -61,17 +61,17 @@ ViewerPane.prototype =
   onViewerConstructed: function(aViewer)
   {
     var old = this.mCurrentViewer;
     this.mCurrentViewer = aViewer;
     var oldEntry = this.mCurrentEntry;
     this.mCurrentEntry = this.mPendingEntry;
 
     this.rebuildViewerContextMenu();
-    
+
     var title = this.mViewerReg.getEntryProperty(this.mCurrentEntry, "description");
     this.setTitle(title);
 
     this.mContainer.onViewerChanged(this, old, oldEntry, this.mCurrentViewer, this.mCurrentEntry);
 
     aViewer.viewee = this.mViewee;
 },
 
@@ -95,34 +95,34 @@ ViewerPane.prototype =
 
   ///////////////////////////////////////////////////////////////////////////
   // Sets the new view to the item just selected from the "viewer list"
   ///////////////////////////////////////////////////////////////////////////
 
   onViewerListCommand: function(aItem)
   {
     this.switchViewer(aItem.entry);
-    
+
   },
 
   ///////////////////////////////////////////////////////////////////////////
   // Prepares the list of viewers for a node, rebuilds the menulist to display
   // them, and load the default viewer for the node.
   //
   // @param Object aObject - the object to begin viewing
   ///////////////////////////////////////////////////////////////////////////
 
   startViewingObject: function(aObject)
   {
     this.mViewee = aObject;
 
     // get the list of viewers which match the node
     var entries = this.mViewerReg.findViewersForObject(aObject);
     this.rebuildViewerList(entries);
-    
+
     if (entries.length > 0 && !this.entryInList(this.mCurrentEntry, entries)) {
       this.switchViewer(entries[0]);
     } else {
       this.mCurrentViewer.viewee = aObject;
     }
   },
 
   ///////////////////////////////////////////////////////////////////////////
@@ -130,27 +130,27 @@ ViewerPane.prototype =
   // for the currently selected node.
   //
   // @param Array aEntries - an array of entries from the viewer registry
   ///////////////////////////////////////////////////////////////////////////
 
   rebuildViewerList: function(aEntries)
   {
     var mpp = this.mListElPopup;
-    
+
     this.mListEl.setAttribute("disabled", aEntries.length <= 0);
 
     // empty the list
     while (mpp.childNodes.length)
       mpp.removeChild(mpp.childNodes[0]);
-    
+
     for (var i = 0; i < aEntries.length; i++) {
       var entry = aEntries[i];
       var menuitem = document.createElement("menuitem");
-      menuitem.setAttribute("value", this.mViewerReg.getEntryProperty(entry, "description"));
+      menuitem.setAttribute("label", this.mViewerReg.getEntryProperty(entry, "description"));
       menuitem.entry = entry;
       mpp.appendChild(menuitem);
     }
   },
 
   ///////////////////////////////////////////////////////////////////////////
   // Loads the viewer described by an entry in the viewer registry.
   //
@@ -161,19 +161,19 @@ ViewerPane.prototype =
   {
     var url = this.mViewerReg.getEntryURL(aEntry);
 
     var loadNew = true;
     if (this.mCurrentViewer) {
       var oldURL = this.mViewerReg.getEntryURL(this.mCurrentEntry);
       if (oldURL == url) {
         loadNew = false;
-      } 
+      }
     }
-    
+
     if (loadNew) {
       this.mPendingEntry = aEntry;
       this.loadViewerURL(url);
     }
   },
 
   ///////////////////////////////////////////////////////////////////////////
   // Begin loading a new viewer from a given url.
@@ -182,17 +182,17 @@ ViewerPane.prototype =
   ///////////////////////////////////////////////////////////////////////////
 
   loadViewerURL: function(aURL)
   {
     if (this.mCurrentViewer) {
       // tell the old viewer it's about to go away
       this.mCurrentViewer.destroy();
     }
-    
+
     // load the new document
     FrameExchange.loadURL(this.mIFrameEl, aURL, this);
   },
 
   ///////////////////////////////////////////////////////////////////////////
   // Rebuild the viewer context menu
   ///////////////////////////////////////////////////////////////////////////
 
@@ -214,17 +214,17 @@ ViewerPane.prototype =
       this.mMenuEl.appendChild(pp);
 
      this.mFormerContextParent = parent;
      this.mContextMenu = pp;
     } else {
       this.mMenuEl.setAttribute("disabled", "true");
     }
   },
-  
+
   ///////////////////////////////////////////////////////////////////////////
   // Check to see if an entry exists in an arry of entries
   //
   // @param nsIRDFResource aEntry - the entry being searched for
   // @param Array aList - array of entries
   ///////////////////////////////////////////////////////////////////////////
 
   entryInList: function(aEntry, aList)
--- a/resources/content/extensions/treeEditable.xml
+++ b/resources/content/extensions/treeEditable.xml
@@ -49,17 +49,17 @@
         <parameter name="aTreecell"/>
         <body><![CDATA[
           // walk up to the treeitem containing the editing cell 
           var item = aTreecell
           while (item && item.localName.toLowerCase() != "treeitem")
             item = item.parentNode;
 
           var editor = this.getEditorElement(aTreecell);
-          editor.setAttribute("value", aTreecell.getAttribute("value"));
+          editor.setAttribute("value", aTreecell.getAttribute("label"));
           aTreecell.parentNode.replaceChild(editor, aTreecell);
 
           this.mEditing = true;
           this.mEditingCell = aTreecell;
           this.mEditingEditor = editor;
           this.mEditingItem = item;
 
           editor.focus();
@@ -71,17 +71,17 @@
       <method name="stopEdit">
         <parameter name="aCancel"/>
         <body><![CDATA[
           if (this.mEditing) {
             var cell = this.mEditingCell;
             var ed = this.mEditingEditor;
             var ev = ed.value;
             var cancel = this.onAfterEdit({ newValue: ev, editingNode: cell, editingItem: this.mEditingItem });
-            if (!aCancel && !cancel) cell.setAttribute("value", ev);
+            if (!aCancel && !cancel) cell.setAttribute("label", ev);
             ed.parentNode.replaceChild(cell, ed);
             this.mEditing = false;
             this.focus();
           }
         ]]></body>
       </method>
 
       <method name="createDefaultEditor">
--- a/resources/content/jsutil/xul/inFormManager.js
+++ b/resources/content/jsutil/xul/inFormManager.js
@@ -1,14 +1,14 @@
 /***************************************************************
 * inFormManager -------------------------------------------------
 *  Manages the reading and writing of forms via simple maps of
 *  attribute/value pairs.  A "form" is simply a XUL window which
 *  contains "form widgets" such as textboxes and menulists.
-* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
+* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 * REQUIRED IMPORTS:
 ****************************************************************/
 
 ////////////////////////////////////////////////////////////////////////////
 //// class inFormManager
 
 var inFormManager =
 {
@@ -22,86 +22,86 @@ var inFormManager =
       el = doc.getElementById(aIds[i]);
       if (el) {
         this.persistIf(doc, el);
         fn = this["read_"+el.localName];
         if (fn)
           map[aIds[i]] = fn(el);
       }
     }
-    
+
     return map;
   },
-  
+
   // void
   writeWindow: function(aWindow, aMap)
   {
     var el, fn;
     var doc = aWindow.document;
     for (var i = 0; i < aIds.length; i++) {
       el = doc.getElementById(aIds[i]);
       if (el) {
         fn = this["write_"+el.localName];
         if (fn)
           fn(el, aMap[aIds[i]]);
       }
     }
   },
-  
+
   persistIf: function(aDoc, aEl)
   {
     if (aEl.getAttribute("persist") == "true" && aEl.hasAttribute("id")) {
       aEl.setAttribute("value", aEl.value);
       aDoc.persist(aEl.getAttribute("id"), "value");
     }
   },
-  
+
   read_textbox: function(aEl)
   {
     return aEl.value;
   },
-  
+
   read_menulist: function(aEl)
   {
-    return aEl.getAttribute("data");
+    return aEl.getAttribute("value");
   },
 
   read_checkbox: function(aEl)
   {
     return aEl.getAttribute("checked") == "true";
   },
 
   read_radiogroup: function(aEl)
   {
-    return aEl.getAttribute("data");
+    return aEl.getAttribute("value");
   },
 
   read_colorpicker: function(aEl)
   {
     return aEl.getAttribute("color");
   },
-  
+
   write_textbox: function(aEl, aValue)
   {
     aEl.setAttribute("value", aValue);
   },
-  
+
   write_menulist: function(aEl, aValue)
   {
-    aEl.setAttribute("data", aValue);
+    aEl.setAttribute("value", aValue);
   },
 
   write_checkbox: function(aEl, aValue)
   {
     aEl.setAttribute("checked", aValue);
   },
 
   write_radiogroup: function(aEl, aValue)
   {
-    aEl.setAttribute("data", aValue);
+    aEl.setAttribute("value", aValue);
   },
 
   write_colorpicker: function(aEl, aValue)
   {
     aEl.color = aValue;
   }
 };
 
--- a/resources/content/jsutil/xul/inTreeTableBuilder.js
+++ b/resources/content/jsutil/xul/inTreeTableBuilder.js
@@ -1,15 +1,15 @@
 /***************************************************************
 * inTreeTableBuilder -------------------------------------------------
 *  Automatically builds up a tree so that it will display a tabular
 *  set of data with titled columns and optionally an icon for each row.
 *  The tree that is supplied must begin with no children other than
 *  a single template element.  The template is required.
-* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
+* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 * REQUIRED IMPORTS:
 *   chrome://inspector/content/jsutil/xul/DNDUtils.js
 ****************************************************************/
 
 //////////// global variables /////////////////////
 
 var inTreeTableBuilderPartCount = 0;
 
@@ -18,22 +18,22 @@ var inTreeTableBuilderPartCount = 0;
 ////////////////////////////////////////////////////////////////////////////
 //// class inTreeTableBuilder
 
 function inTreeTableBuilder(aTree, aNameSpace, aArcName)
 {
   this.tree = aTree;
   this.nameSpace = aNameSpace;
   this.arcName = aArcName;
-  
+
   this.mColumns = [];
   this.mExtras = [];
 }
 
-inTreeTableBuilder.prototype = 
+inTreeTableBuilder.prototype =
 {
   mTree: null,
   // datasource stuff
   mNameSpace: null,
   mArcName: null,
   mIsRefContainer: true,
   mIsContainer: false,
 
@@ -41,44 +41,44 @@ inTreeTableBuilder.prototype =
   mIsIconic: false,
   mColumns: null,
   mSplitters: true,
   mItemFields: null,
   mItemAttrs: null,
   mAllowDND: false,
   mLastDragCol: null,
   mLastDragColWhere: null,
-  
+
   ////////////////////////////////////////////////////////////////////////////
   //// properties
-  
+
   // the xul tree node we will construct
   get tree() { return this.mTree },
   set tree(aVal) { this.mTree = aVal; aVal._treeBuilder = this },
 
   // the namespace to use for all fields
   get nameSpace() { return this.mNameSpace },
   set nameSpace(aVal) { this.mNameSpace = aVal },
-  
+
   // the name of the arc to the container
   get arcName() { return this.mArcName },
   set arcName(aVal) { this.mArcName = aVal },
 
   // is the datasource ref an arc to a container, or a container itself
   get isRefContainer() { return this.mIsRefContainer },
   set isRefContainer(aVal) { this.mIsRefContainer = aVal },
-  
+
   // is each row a potential container?
   get isContainer() { return this.mIsContainer },
   set isContainer(aVal) { this.mIsContainer = aVal },
 
   // place an icon before the first column of each row
   get isIconic() { return this.mIsIconic },
   set isIconic(aVal) { this.mIsIconic = aVal },
-  
+
   // put splitters between the columns?
   get useSplitters() { return this.mSplitters },
   set useSplitters(aVal) { this.mSplitters = aVal },
 
   // extra attributes to set on the treeitem
   get itemAttributes() { return this.mItemAttrs },
   set itemAttributes(aVal) { this.mItemAttrs = aVal },
 
@@ -87,342 +87,342 @@ inTreeTableBuilder.prototype =
   set itemFields(aVal) { this.mItemFields = aVal },
 
   // extra data fields to set on the treeitem
   get allowDragColumns() { return this.mAllowDND },
   set allowDragColumns(aVal) { this.mAllowDND = aVal },
 
   ////////////////////////////////////////////////////////////////////////////
   //// event handlers
-  
+
   get onColumnAdd() { return this.mOnColumnAdd },
   set onColumnAdd(aFn) { this.mOnColumnAdd = aFn },
 
   get onColumnRemove() { return this.mOnColumnRemove },
   set onColumnRemove(aFn) { this.mOnColumnRemove = aFn },
 
   ////////////////////////////////////////////////////////////////////////////
   //// initialization
 
   initialize: function()
   {
     this.initColumns();
     this.initTemplate();
   },
-  
+
   initColumns: function()
   {
     // initialize the treehead
     var treehead = document.createElementNS(kXULNSURI, "treehead");
     var treerow = document.createElementNS(kXULNSURI, "treerow");
     this.mTreeHeadRow = treerow;
-    
+
     treehead.appendChild(treerow);
     this.mTree.appendChild(treehead);
-    
+
     // initialize the treecolgroup
     var colgroup = document.createElementNS(kXULNSURI, "treecolgroup");
     this.mColGroup = colgroup;
     this.mTree.appendChild(colgroup);
-    
+
     this.initDND();
   },
-  
+
   initTemplate: function()
   {
     var template = this.mTree.getElementsByTagNameNS(kXULNSURI, "template")[0];
     this.mTemplate = template;
     this.clearChildren(template);
-    
+
     var rule = document.createElementNS(kXULNSURI, "rule");
     template.appendChild(rule);
     this.mRule = rule;
-    
+
     this.createDefaultConditions();
-    
+
     var bindings = document.createElementNS(kXULNSURI, "bindings");
     rule.appendChild(bindings);
     this.mBindings = bindings;
-    
+
     this.createDefaultAction();
   },
-  
+
   createDefaultConditions: function()
   {
     var conditions = document.createElementNS(kXULNSURI, "conditions");
     this.mRule.appendChild(conditions);
 
     var content = document.createElementNS(kXULNSURI, "content");
     content.setAttribute("uri", "?uri");
     conditions.appendChild(content);
 
-    var triple = this.createTriple("?uri", this.mNameSpace+this.mArcName, 
+    var triple = this.createTriple("?uri", this.mNameSpace+this.mArcName,
       this.mIsRefContainer ? "?table" : "?item");
     conditions.appendChild(triple);
 
     if (this.mIsRefContainer) {
       var member = this.createMember("?table", "?item");
       conditions.appendChild(member);
     }
   },
-  
+
   createDefaultAction: function()
   {
     var action = document.createElementNS(kXULNSURI, "action");
     this.mRule.appendChild(action);
-    
+
     var treechildren = this.createTemplatePart("treechildren");
     action.appendChild(treechildren);
     treechildren.setAttribute("flex", "1");
     var treeitem = this.createTemplatePart("treeitem");
     treechildren.appendChild(treeitem);
     treeitem.setAttribute("uri", "?item");
     this.mTreeItem = treeitem;
     var treerow = this.createTemplatePart("treerow");
     treeitem.appendChild(treerow);
     this.mTreeRow = treerow;
 
     // assign the item attributes
     if (this.mItemAttrs)
       for (key in this.mItemAttrs)
         treeitem.setAttribute(key, this.mItemAttrs[key]);
   },
-  
+
   createDefaultBindings: function()
   {
     // assign the item fields
     var binding;
     if (this.mItemFields) {
       for (key in this.mItemFields) {
         binding = this.createBinding(this.mItemFields[key]);
         this.mBindings.appendChild(binding);
         this.mTreeItem.setAttribute(key, "?" + this.mItemFields[key]);
       }
     }
   },
-  
+
   createTriple: function(aSubject, aPredicate, aObject)
   {
     var triple = document.createElementNS(kXULNSURI, "triple");
     triple.setAttribute("subject", aSubject);
     triple.setAttribute("predicate", aPredicate);
     triple.setAttribute("object", aObject);
-    
+
     return triple;
   },
-  
+
   createMember: function(aContainer, aChild)
   {
     var member = document.createElementNS(kXULNSURI, "member");
     member.setAttribute("container", aContainer);
     member.setAttribute("child", aChild);
-    
+
     return member;
   },
-  
+
   reset: function()
   {
     this.mColumns = [];
     this.mIsIconic = false;
-    
+
     this.resetTree();
   },
-  
+
   resetTree: function()
   {
     this.clearChildren(this.mColGroup);
     this.clearChildren(this.mTreeHeadRow);
     this.clearChildren(this.mBindings);
     this.clearChildren(this.mTreeRow);
     this.createDefaultBindings();
   },
-  
+
   clearChildren: function(aEl)
   {
     while (aEl.childNodes.length)
       aEl.removeChild(aEl.lastChild);
   },
-  
+
   ////////////////////////////////////////////////////////////////////////////
   //// column drag and drop
 
   initDND: function()
   {
     if (this.mAllowDND) {
       // addEventListener doesn't work for dnd events, apparently... so we use the attributes
       this.addDNDListener(this.mTree, "ondragenter");
       this.addDNDListener(this.mTree, "ondragover");
       this.addDNDListener(this.mTree, "ondragexit");
       this.addDNDListener(this.mTree, "ondraggesture");
       this.addDNDListener(this.mTree, "ondragdrop");
     }
-  },  
-  
+  },
+
   onDragEnter: function(aEvent)
   {
   },
-  
+
   onDragOver: function(aEvent)
   {
     if (!DNDUtils.checkCanDrop("TreeTableBuilder/column-add"))
       return;
-      
+
     var idx = this.getColumnIndexFromX(aEvent.clientX, 50);
     this.mColumnInsertIndex = idx;
-    var kids = this.mColGroup.childNodes; 
+    var kids = this.mColGroup.childNodes;
     var col = this.getColumnAt(idx);
     if (idx == -1)
       this.markColumnInsert(col, "after", idx);
     else
       this.markColumnInsert(col, "before", idx);
   },
-  
+
   onDragExit: function(aEvent)
   {
   },
-  
+
   onDragDrop: function(aEvent)
   {
     this.markColumnInsert(null);
     var dragService = XPCU.getService("@mozilla.org/widget/dragservice;1", "nsIDragService");
     var dragSession = dragService.getCurrentSession();
-    
+
     if (!dragSession.isDataFlavorSupported("TreeTableBuilder/column-add"))
       return false;
-      
+
     var trans = XPCU.createInstance("@mozilla.org/widget/transferable;1", "nsITransferable");
     trans.addDataFlavor("TreeTableBuilder/column-add");
-  
+
     dragSession.getData(trans, 0);
     var data = new Object();
     trans.getAnyTransferData ({}, data, {});
-    
+
     var string = XPCU.QI(data.value, "nsISupportsWString");
-    
-    this.insertColumn(this.mColumnInsertIndex, 
+
+    this.insertColumn(this.mColumnInsertIndex,
       {
-        name: string.data, 
+        name: string.data,
         title: string.data,
         flex: 1
       });
     this.build();
     // if we rebuildContent during this thread, it will crash in the dnd service
     setTimeout(function(aTreeBuilder) { aTreeBuilder.buildContent() }, 1, this);
-    
+
     // bug 56270 - dragSession.sourceDocument is null --
     // causes me to code this very temporary, very nasty hack
     // to tell columnsDialog.js about the drop
     if (this.mTree.onClientDrop) {
       this.mTree.onClientDrop();
     }
   },
-  
+
   markColumnInsert: function (aColumn, aWhere, aIdx)
   {
     var col = this.mLastDragCol;
     var lastWhere = this.mLastDragColWhere;
     if (col)
       col.setAttribute("dnd-insert-"+lastWhere, "false");
-    
+
     if (aWhere != "before" && aWhere != "after") {
       this.mLastDragCol = null;
       this.mLastDragColWhere = null;
       return;
     }
-      
+
     col = aColumn;
     if (col) {
       this.mLastDragCol = col;
       this.mLastDragColWhere = aWhere;
       col.setAttribute("dnd-insert-"+aWhere, "true");
     }
   },
-  
+
   getColumnIndexFromX: function(aX, aThresh)
   {
     var colgroup = this.mColGroup;
     var width = 0, cw;
     var col;
     var pct = aThresh/100;
     for (var i = 0; i < this.columnCount; ++i) {
       col = this.getColumnAt(i);
       cw = col.boxObject.width;
       width += cw;
       if (width-(cw*pct) > aX)
         return i;
     }
-    
+
     return -1;
   },
 
   getColumnIndexFromHeader: function(aHeader)
   {
     var headers = this.mTreeHeadRow.childNodes;
     for (var i = 0; i < headers.length; ++i) {
       if (headers[i] == aHeader)
-        return i;      
+        return i;
     }
     return -1;
   },
-  
+
   //// for drag-n-drop removal/arrangement of columns
-  
+
   onDragGesture: function(aEvent)
   {
     var target = aEvent.target;
     if (target.parentNode == this.mTreeHeadRow) {
-      var column = target.getAttribute("value");
-      
+      var column = target.getAttribute("label");
+
       var idx = this.getColumnIndexFromHeader(target);
       if (idx == -1) return;
       this.mColumnDragging = idx;
-      
+
       DNDUtils.invokeSession(target, ["TreeTableBuilder/column-remove"], [column]);
     }
   },
-  
+
   addColumnDropTarget: function(aBox)
-  { 
+  {
     aBox._treeBuilderDropTarget = this;
     this.addDNDListener(aBox, "ondragover", "Target");
     this.addDNDListener(aBox, "ondragdrop", "Target");
   },
-  
+
   removeColumnDropTarget: function(aBox)
   {
     aBox._treeBuilderDropTarget = this;
     this.removeDNDListener(aBox, "ondragover", "Target");
     this.removeDNDListener(aBox, "ondragdrop", "Target");
   },
-  
+
   onDragOverTarget: function(aBox, aEvent)
   {
     DNDUtils.checkCanDrop("TreeTableBuilder/column-remove");
   },
-  
+
   onDragDropTarget: function(aBox, aEvent)
   {
     this.removeColumn(this.mColumnDragging);
     this.build();
     // if we rebuildContent during this thread, it will crash in the dnd service
     setTimeout(function(aTreeBuilder) { aTreeBuilder.buildContent() }, 1, this);
   },
 
   // these are horrible hacks to compensate for the lack of true multiple
   // listener support for the DND events
-  
+
   addDNDListener: function(aBox, aType, aModifier)
   {
    var js = "inTreeTableBuilder_"+aType+(aModifier?"_"+aModifier:"")+"(this, event);";
 
    var attr = aBox.getAttribute(aType);
    attr = attr ? attr : "";
    aBox.setAttribute(aType, attr + js);
   },
-  
+
   removeDNDListener: function(aBox, aType, aModifier)
   {
    var js = "inTreeTableBuilder_"+aType+(aModifier?"_"+aModifier:"")+"(this, event);";
 
    var attr = aBox.getAttribute(aType);
    var idx = attr.indexOf(js);
    if (idx != -1)
      attr = attr.substr(0,idx) + attr.substr(idx+1);
@@ -430,58 +430,58 @@ inTreeTableBuilder.prototype =
   },
 
   ////////////////////////////////////////////////////////////////////////////
   //// column properties
 
   addColumn: function(aData)
   {
     this.mColumns.push(aData);
-    
+
     if (this.mOnColumnAdd)
       this.mOnColumnAdd(this.mColumns.length-1)
   },
-  
+
   insertColumn: function(aIndex, aData)
   {
     var idx;
     if (aIndex == -1) {
       this.mColumns.push(aData);
       idx = this.mColumns.length-1;
     } else {
       this.mColumns.splice(aIndex, 0, aData);
       idx = aIndex;
     }
 
     if (this.mOnColumnAdd)
       this.mOnColumnAdd(idx);
   },
-    
+
   removeColumn: function(aIndex)
   {
     this.mColumns.splice(aIndex, 1);
 
     if (this.mOnColumnRemove)
       this.mOnColumnRemove(aIndex);
   },
-  
+
   getColumnAt: function(aIndex)
   {
     var idx;
     var kids = this.mColGroup.childNodes;
     if (aIndex == -1)
       idx = kids.length-1;
     else {
       // step by 2 to skip over splitters, if they are used
       var step = this.mSplitters ? 2 : 1;
       idx = aIndex*step;
     }
-    return kids[idx]   
+    return kids[idx]
   },
-  
+
   get columnCount() { return this.mColumns.length },
 
   getColumnName: function(aIndex) { return this.mColumns[aIndex].name  },
   setColumnName: function(aIndex, aValue) { this.mColumns[aIndex].name = aValue },
 
   getColumnTitle: function(aIndex) { return this.mColumns[aIndex].title  },
   setColumnTitle: function(aIndex, aValue) { this.mColumns[aIndex].title = aValue },
 
@@ -506,28 +506,28 @@ inTreeTableBuilder.prototype =
       this.resetTree();
       this.buildColGroup();
       this.buildTreeHead();
       this.buildTemplate();
       if (aBuildContent)
         this.buildContent();
     } catch (ex) {
       debug("### ERROR - inTreeTableBuilder::build failed.\n" + ex);
-    } 
+    }
 
     //if ("dumpDOM" in window)
       //dumpDOM(this.mTemplate);
-    
+
   },
-  
+
   buildContent: function()
   {
     this.mTree.builder.rebuild();
   },
-  
+
   buildColGroup: function()
   {
     var cols = this.mColumns;
     var colgroup = this.mColGroup;
     var col, val, split;
     for (var i = 0; i < cols.length; i++) {
       col = document.createElementNS(kXULNSURI, "treecol");
       col.setAttribute("id", "treecol-"+cols[i].name);
@@ -543,89 +543,89 @@ inTreeTableBuilder.prototype =
 
       if (this.mSplitters && i < cols.length-1) {
         split = document.createElementNS(kXULNSURI, "splitter");
         split.setAttribute("class", "tree-splitter");
         colgroup.appendChild(split);
       }
     }
   },
-  
+
   buildTreeHead: function()
   {
     var cols = this.mColumns;
     var row = this.mTreeHeadRow;
     var cell, val;
     for (var i = 0; i < cols.length; i++) {
       cell = document.createElementNS(kXULNSURI, "treecell");
       cell.setAttribute("class", "treecell-header");
       val = cols[i].title;
       if (val)
-        cell.setAttribute("value", val);
+        cell.setAttribute("label", val);
       row.appendChild(cell);
     }
   },
-  
+
   buildTemplate: function()
   {
     var cols = this.mColumns;
     var bindings = this.mBindings;
     var row = this.mTreeRow;
     var cell, binding, val, extras, attrs, key, className;
-    
+
     if (this.mIsIconic) {
       binding = this.createBinding("_icon");
       bindings.appendChild(binding);
     }
-    
+
     for (var i = 0; i < cols.length; ++i) {
       val = cols[i].name;
-      if (!val) 
+      if (!val)
         throw "Column data is incomplete - missing name at index " + i + ".";
 
       cell = this.createTemplatePart("treecell");
       className = "";
-      
+
       // build the icon data field
       if (i == 0 && this.mIsIconic) {
         className += "treecell-iconic ";
         cell.setAttribute("src", "?_icon");
       }
-      
+
       // mark first node as a container, if necessary
       if (i == 0 && this.mIsContainer)
         className += "treecell-indent";
-      
+
       // build the default value data field
       binding = this.createBinding(val);
       bindings.appendChild(binding);
-      cell.setAttribute("value", "?" + val);
+      cell.setAttribute("label", "?" + val);
 
       cell.setAttribute("class", className);
       row.appendChild(cell);
     }
-  }, 
-  
+  },
+
   createBinding: function(aName)
   {
     var binding = document.createElementNS(kXULNSURI, "binding");
     binding.setAttribute("subject", "?item");
     binding.setAttribute("predicate", this.mNameSpace+aName);
     binding.setAttribute("object", "?" + aName);
     return binding;
   },
-  
+
   createTemplatePart: function(aTagName)
   {
     var el = document.createElementNS(kXULNSURI, aTagName);
     el.setAttribute("id", "templatePart"+inTreeTableBuilderPartCount);
     inTreeTableBuilderPartCount++;
     return el;
   }
-    
+
 };
 
 function inTreeTableBuilder_ondraggesture(aTree, aEvent)
 {
   return aTree._treeBuilder.onDragGesture(aEvent);
 }
 
 function inTreeTableBuilder_ondragenter(aTree, aEvent)
--- a/resources/content/popupOverlay.xul
+++ b/resources/content/popupOverlay.xul
@@ -6,62 +6,62 @@
 ]>
 
 <overlay id="ovPopupsMain"
          xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 
   <popupset id="ppsGlobalPopupset">
     <popup id="ppSearchResults">
       <!--
-        <menuitem id="ppSearchResults-view" value="&cmdViewSearchItem.label;" observes="cmdViewSearchItem"/>
-        <menuitem id="ppSearchResults-edit" value="&cmdEditSearchItem.label;" observes="cmdEditSearchItem"/>
+        <menuitem id="ppSearchResults-view" label="&cmdViewSearchItem.label;" observes="cmdViewSearchItem"/>
+        <menuitem id="ppSearchResults-edit" label="&cmdEditSearchItem.label;" observes="cmdEditSearchItem"/>
       -->
       <menuseparator id="ppSearchResults-insertion"/>
-      <menuitem id="ppSearchResults-copyLine" value="&cmdCopySearchItemLine.label;" observes="cmdCopySearchItemLine"/>
-      <menuitem id="ppSearchResults-copyAll" value="&cmdCopySearchItemAll.label;" observes="cmdCopySearchItemAll"/>
-      <menuitem id="ppSearchResults-save" value="&cmdSaveSearchItemText.label;" observes="cmdSaveSearchItemText"/>
+      <menuitem id="ppSearchResults-copyLine" label="&cmdCopySearchItemLine.label;" observes="cmdCopySearchItemLine"/>
+      <menuitem id="ppSearchResults-copyAll" label="&cmdCopySearchItemAll.label;" observes="cmdCopySearchItemAll"/>
+      <menuitem id="ppSearchResults-save" label="&cmdSaveSearchItemText.label;" observes="cmdSaveSearchItemText"/>
       <menuseparator/>
-      <menuitem id="ppSearchResults-clear" value="&cmdClearSearch.label;" observes="cmdClearSearch"/>
-      <menuitem id="ppSearchResults-hide" value="&cmdHideSearchItems.label;" observes="cmdToggleSearch"/>
+      <menuitem id="ppSearchResults-clear" label="&cmdClearSearch.label;" observes="cmdClearSearch"/>
+      <menuitem id="ppSearchResults-hide" label="&cmdHideSearchItems.label;" observes="cmdToggleSearch"/>
     </popup>
   </popupset>
 
   <menupopup id="mppFile">
-    <menuitem value="&cmdShowOpenURLDialog.label;..." accesskey="&cmdShowOpenURLDialog.accesskey;" observes="cmdShowOpenURLDialog"/>
+    <menuitem label="&cmdShowOpenURLDialog.label;..." accesskey="&cmdShowOpenURLDialog.accesskey;" observes="cmdShowOpenURLDialog"/>
     <menuseparator/>
-    <menuitem value="&cmdExit.label;" accesskey="&cmdExit.accesskey;" observes="cmdExit"/>
+    <menuitem label="&cmdExit.label;" accesskey="&cmdExit.accesskey;" observes="cmdExit"/>
   </menupopup>
 
   <menupopup id="mppEdit">
-    <menuitem value="&cmdUndo.label;" accesskey="&cmdUndo.accesskey;" observes="cmdUndo"/>
-    <menuitem value="&cmdRedo.label;" accesskey="&cmdRedo.accesskey;" observes="cmdRedo"/>
+    <menuitem label="&cmdUndo.label;" accesskey="&cmdUndo.accesskey;" observes="cmdUndo"/>
+    <menuitem label="&cmdRedo.label;" accesskey="&cmdRedo.accesskey;" observes="cmdRedo"/>
     <menuseparator/>
-    <menuitem value="&cmdCut.label;" accesskey="&cmdCut.accesskey;" observes="cmdCut"/>
-    <menuitem value="&cmdCopy.label;" accesskey="&cmdCopy.accesskey;" observes="cmdCopy"/>
-    <menuitem value="&cmdPaste.label;" accesskey="&cmdPaste.accesskey;" observes="cmdPaste"/>
-    <menuitem value="&cmdDelete.label;" accesskey="&cmdDelete.accesskey;" observes="cmdDelete"/>
+    <menuitem label="&cmdCut.label;" accesskey="&cmdCut.accesskey;" observes="cmdCut"/>
+    <menuitem label="&cmdCopy.label;" accesskey="&cmdCopy.accesskey;" observes="cmdCopy"/>
+    <menuitem label="&cmdPaste.label;" accesskey="&cmdPaste.accesskey;" observes="cmdPaste"/>
+    <menuitem label="&cmdDelete.label;" accesskey="&cmdDelete.accesskey;" observes="cmdDelete"/>
     <menuseparator/>
-    <menuitem value="&cmdPrefs.label;..." accesskey="&cmdPrefs.accesskey;" observes="cmdShowPrefsDialog"/>
+    <menuitem label="&cmdPrefs.label;..." accesskey="&cmdPrefs.accesskey;" observes="cmdShowPrefsDialog"/>
   </menupopup>
 
   <menupopup id="mppSearch" oncreate="inspector.initPopup(this)">
     <menuseparator/>
-    <menu value="&mnSearchPlugins.label;" accesskey="&mnSearchPlugins.accesskey;">
+    <menu label="&mnSearchPlugins.label;" accesskey="&mnSearchPlugins.accesskey;">
       <menupopup id="mppSearchPlugins">
-        <menuitem value="&cmdRunSearch.label;" accesskey="&cmdRunSearch.accesskey;" observes="cmdRunSearch"/>
+        <menuitem label="&cmdRunSearch.label;" accesskey="&cmdRunSearch.accesskey;" observes="cmdRunSearch"/>
       </menupopup>
     </menu>
   </menupopup>
 
   <menupopup id="mppView" >
-    <menuitem class="menuitem-iconic" type="checkbox" value="&cmdToggleBrowser.label;" observes="cmdToggleBrowser"/>
-    <menuitem class="menuitem-iconic" type="checkbox" value="&cmdToggleSearch.label;" observes="cmdToggleSearch"/>
+    <menuitem class="menuitem-iconic" type="checkbox" label="&cmdToggleBrowser.label;" observes="cmdToggleBrowser"/>
+    <menuitem class="menuitem-iconic" type="checkbox" label="&cmdToggleSearch.label;" observes="cmdToggleSearch"/>
     <menuseparator/>
-    <menuitem class="menuitem-iconic" type="checkbox" value="&cmdFlashSelected.label;" observes="cmdFlashSelected"/>
+    <menuitem class="menuitem-iconic" type="checkbox" label="&cmdFlashSelected.label;" observes="cmdFlashSelected"/>
   </menupopup>
 
   <menupopup id="mppHelp">
-    <menuitem value="&cmdNotes.label;" accesskey="&cmdNotes.accesskey;" observes="cmdShowNotes"/>
+    <menuitem label="&cmdNotes.label;" accesskey="&cmdNotes.accesskey;" observes="cmdShowNotes"/>
     <menuseparator/>
-    <menuitem value="&cmdAbout.label;" accesskey="&cmdAbout.accesskey;" observes="cmdShowAbout"/>
+    <menuitem label="&cmdAbout.label;" accesskey="&cmdAbout.accesskey;" observes="cmdShowAbout"/>
   </menupopup>
 
 </overlay>
--- a/resources/content/prefs/pref-inspector.xul
+++ b/resources/content/prefs/pref-inspector.xul
@@ -27,17 +27,17 @@
     AddColorPicker(initFunction);
 
   ]]/>
 
   <box class="box-smallheader" title="&Inspector.label;"/>
 
   <box id="bxGeneralPrefs">
     <titledbox>
-      <title><text value="Element Blinking"/></title>
+      <label value="Element Blinking"/>
       <box orient="vertical">
         <grid>
           <columns>
             <column/>
             <column flex="1"/>
           </columns>
           <rows>
             <row autostretch="never">
@@ -69,15 +69,15 @@
                 <textbox id="txfElSpeed" style="width: 4em"
                   pref="true" preftype="int" prefstring="inspector.blink.speed" prefattribute="value"/>
                 <text value="(ms)"/>
                </box>
             </row>
           </rows>
         </grid>
         <box autostretch="never">
-          <checkbox id="cbElOn" value="Blink Selected Element"
+          <checkbox id="cbElOn" label="Blink Selected Element"
               pref="true" preftype="bool" prefstring="inspector.blink.on" prefattribute="checked"/>
         </box>
       </box>
     </titledbox>
   </box>
 </window>
--- a/resources/content/prefs/pref-sidebar.xul
+++ b/resources/content/prefs/pref-sidebar.xul
@@ -15,12 +15,12 @@
   <box class="box-smallheader" title="&Sidebar.label;"/>
 
   <box id="bxSidebarPrefs">
     <box id="bxSidebarInstalled" collapsed="true" flex="1">
       <text value="The sidebar panel is currently installed."/>
     </box>
     <box id="bxSidebarNotInstalled" collapsed="true" flex="1" orient="vertical">
       <text value="The sidebar panel is not currently installed.  Click the 'Install' button to add it to your sidebar."/>
-      <button value="Install" onclick="gSidebarPrefs.installSidebar()"/>
+      <button label="Install" onclick="gSidebarPrefs.installSidebar()"/>
     </box>
   </box>
 </window>
--- a/resources/content/prefs/prefsOverlay.xul
+++ b/resources/content/prefs/prefsOverlay.xul
@@ -3,22 +3,22 @@
 <!DOCTYPE overlay SYSTEM "chrome://inspector/locale/prefs.dtd">
 
 <overlay id="ovPrefs" 
         xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
   
   <treechildren id="panelChildren">
     <treeitem id="inspector" container="true" open="true">
       <treerow>       
-        <treecell class="treecell-indent" url="chrome://inspector/content/prefs/pref-inspector.xul" value="&Inspector.label;"/> 
+        <treecell class="treecell-indent" url="chrome://inspector/content/prefs/pref-inspector.xul" label="&Inspector.label;"/> 
       </treerow>
       <treechildren id="inspectorChildren">
         <treeitem>
           <treerow>
-            <treecell class="treecell-indent" url="chrome://inspector/content/prefs/pref-sidebar.xul" value="&Sidebar.label;"/> 
+            <treecell class="treecell-indent" url="chrome://inspector/content/prefs/pref-sidebar.xul" label="&Sidebar.label;"/> 
           </treerow>
         </treeitem>
       </treechildren>
     </treeitem>
   </treechildren>
 
 </overlay>
 
--- a/resources/content/search/modules/findFiles/dialog.xul
+++ b/resources/content/search/modules/findFiles/dialog.xul
@@ -20,17 +20,17 @@
   <box orient="vertical">
     <box autostretch="never">
       <text id="txSearchFor" value="&txSearchFor.label;"/>
       <textbox id="txfSearchFor" style="width: 25em;" persist="true"/>
     </box>
     <box autostretch="never">
       <text id="txSearchPath" value="&txSearchPath.label;"/>
       <textbox id="txfSearchPath" style="width: 25em;" persist="true"/>
-      <button id="btnSearchPathBrowser" value="Browse" onclick="dialog.browse()" persist="true"/>
+      <button id="btnSearchPathBrowser" label="Browse" onclick="dialog.browse()" persist="true"/>
     </box>
     <box autostretch="never">
-      <checkbox id="cbRecurse" value="&cbRecurse.label;"/>
+      <checkbox id="cbRecurse" label="&cbRecurse.label;"/>
     </box>    
   </box>
 
   <box id="okCancelButtonsRight"/>
 </window>
--- a/resources/content/search/modules/junkImgs/dialog.xul
+++ b/resources/content/search/modules/junkImgs/dialog.xul
@@ -25,15 +25,15 @@
       <button id="btnSearchPathBrowser" value="Browse" onclick="dialog.browse()"/>
     </box>
     <!--
     <box autostretch="never">
       <text id="txRemotePath" value="Target URL:"/>
       <textbox id="txfRemotePath" flex="1" persist="true"/>
     </box>
     <box autostretch="never">
-      <checkbox id="cbIsSkin" value="Target is a Skin" checked="true" persist="true"/>
+      <checkbox id="cbIsSkin" label="Target is a Skin" checked="true" persist="true"/>
     </box>
     -->
   </box>
 
   <box id="okCancelButtonsRight"/>
 </window>
--- a/resources/content/search/modules/junkImgs/junkImgs.xml
+++ b/resources/content/search/modules/junkImgs/junkImgs.xml
@@ -9,18 +9,18 @@
   <dialog elements="txfSearchPath" resizable="true"
       href="chrome://inspector/content/search/modules/junkImgs/dialog.xul"/>
 
   <columns delimiter="">
     <column title="Path" name="path" flex="1" copy="true"/>
   </columns>
 
   <contextmenu>
-    <xul:menuitem value="View..." oncommand="this.search.cmdViewItem()" isEnabled="return true"/>
-    <xul:menuitem value="Delete File" oncommand="this.search.cmdDeleteItems()" isEnabled="return true"/>
+    <xul:menuitem label="View..." oncommand="this.search.cmdViewItem()" isEnabled="return true"/>
+    <xul:menuitem label="Delete File" oncommand="this.search.cmdDeleteItems()" isEnabled="return true"/>
   </contextmenu>
 
   <implementation>
     
     <method name="constructor">
       <body><![CDATA[
         mHashTable = null;
       ]]></body>
--- a/resources/content/search/modules/popupOverlay.xul
+++ b/resources/content/search/modules/popupOverlay.xul
@@ -6,21 +6,21 @@
 ]>
 
 <overlay id="ovJunkImgsPopup"
         xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
   
   <menupopup id="mppSearchPlugins">
   
     <menuitem id="item:startFindFiles" position="1"
-      value="&findFiles.title;..."
+      label="&findFiles.title;..."
       observes="cmd:startFindFiles"/>
 
     <menuitem id="item:startJunkImgs" position="2"
-      value="&junkImgs.title;..."
+      label="&junkImgs.title;..."
       isDisabled="return !inspector.isViewingContent"
       observes="cmd:startJunkImgs"/>
     
     <menuseparator position="3"/>
     
   </menupopup>
 
 </overlay>
--- a/resources/content/sidebar/sidebar.xul
+++ b/resources/content/sidebar/sidebar.xul
@@ -34,18 +34,18 @@
     </popup>
   </popupset>
   
   <popupset id="ppsGlobalPopupset" collapsed="true"/>
   <broadcasterset id="brsGlobalCommands"/>
 
   <tabcontrol id="tbcInspectorSidebar" flex="1">
     <tabbox>
-      <tab value="Document"/>
-      <tab value="Node"/>
+      <tab label="Document"/>
+      <tab label="Node"/>
     </tabbox>
     <tabpanel flex="1">
       <box id="bxDocPane" flex="1" orient="vertical">
         <box id="bxDocHeader" autostretch="never">
           <menubutton class="viewer-list" ins-role="viewer-list" disabled="true"/>
           <text class="viewer-title" ins-role="viewer-title" flex="1"/>
           <menubutton class="viewer-menu" ins-role="viewer-menu" disabled="true"/>
         </box>
--- a/resources/content/tasksOverlay.xul
+++ b/resources/content/tasksOverlay.xul
@@ -4,23 +4,23 @@
 
 <overlay id="eInspectorTasksOverlay"
          xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 
   <script type="application/x-javascript">
     
     function launchInspector()
     {
-    	toOpenWindowByType("inspector", "chrome://inspector/content");
+      toOpenWindowByType("inspector", "chrome://inspector/content");
     }
 
   </script>
 
   <broadcasterset id="broadcasterset">
-    <broadcaster id="Tasks:Inspector" value="&inspectorTaskCmd.label;" oncommand="launchInspector();"/>
+    <broadcaster id="Tasks:Inspector" label="&inspectorTaskCmd.label;" oncommand="launchInspector();"/>
   </broadcasterset>
 
 <!--
   <keyset id="keyset">
     <key id="kInspectorTask" key="&inspectorTaskCmd.commandkey;" observes="Tasks:Inspector" modifiers="accel"/>
   </keyset>
 -->
   <menupopup id="toolsPopup">
--- a/resources/content/toolboxOverlay.xul
+++ b/resources/content/toolboxOverlay.xul
@@ -10,45 +10,45 @@
 <overlay id="ovToolbox"
          xmlns:ins="http://www.mozilla.org/inspector#"
          xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 
   <popupset id="ppsGlobalPopupset"/>
 
   <menubar id="mbrInspectorMain">
 
-    <menu id="mnFile" value="&mnFile.label;" accesskey="&mnFile.accesskey;">
+    <menu id="mnFile" label="&mnFile.label;" accesskey="&mnFile.accesskey;">
       <menupopup id="mppFile">
         <menu 
-          id="mnWindowsFile" value="&mnWindows.label;" accesskey="&mnWindows.accesskey;"
+          id="mnWindowsFile" label="&mnWindows.label;" accesskey="&mnWindows.accesskey;"
           datasources="rdf:window-mediator" ref="NC:WindowMediatorRoot">
           <template>
             <rule>
               <menupopup>
-                <menuitem uri="rdf:*" value="&NC-rdf;Name" oncommand="inspector.goToWindow(this)"/>
+                <menuitem uri="rdf:*" label="&NC-rdf;Name" oncommand="inspector.goToWindow(this)"/>
               </menupopup>
             </rule>
           </template>
         </menu>
       </menupopup>
     </menu>
 
-    <menu id="mnEdit" value="&mnEdit.label;" accesskey="&mnEdit.accesskey;">
+    <menu id="mnEdit" label="&mnEdit.label;" accesskey="&mnEdit.accesskey;">
       <menupopup id="mppEdit"/>
     </menu>
 
-    <menu id="mnSearch" value="&mnSearch.label;" accesskey="&mnSearch.accesskey;">
+    <menu id="mnSearch" label="&mnSearch.label;" accesskey="&mnSearch.accesskey;">
       <menupopup id="mppSearch"/>
     </menu>
 
-    <menu id="mnView" value="&mnView.label;" accesskey="&mnView.accesskey;">
+    <menu id="mnView" label="&mnView.label;" accesskey="&mnView.accesskey;">
       <menupopup id="mppView"/>
     </menu>
 
-    <menu id="mnHelp" value="&mnHelp.label;" accesskey="&mnHelp.accesskey;">
+    <menu id="mnHelp" label="&mnHelp.label;" accesskey="&mnHelp.accesskey;">
       <menupopup id="mppHelp"/>
     </menu>
 
   </menubar>
 
   <toolbar id="tbInspectorPrimary">
 
     <box id="bxURLBar" flex="1" autostretch="never" valign="middle">
@@ -74,24 +74,24 @@
               <rule>
                 <conditions>
                   <content uri="?uri"/>
                   <member container="?uri" child="?item"/>
                   <triple subject="?item" predicate="&INS-rdf;URL" object="?url"/>
                 </conditions>
 
                 <action>
-                  <menuitem uri="?item" value="?url" oncommand="inspector.goToHistoryItem(this)"/>
+                  <menuitem uri="?item" label="?url" oncommand="inspector.goToHistoryItem(this)"/>
                 </action>
               </rule>
             </template>
           
           </menupopup>
 
         </menubutton>
       </box>
 
-      <button class="button-toolbar-3" value="&btnInspect.label;" observes="cmdGotoURL"/>
+      <button class="button-toolbar-3" label="&btnInspect.label;" observes="cmdGotoURL"/>
     </box>
 
   </toolbar>
 
 </overlay>
--- a/resources/content/viewers/dom/dom.xul
+++ b/resources/content/viewers/dom/dom.xul
@@ -32,18 +32,18 @@
       isvalid="return viewer.cmdInspectBrowserIsValid()"
       oncommand="viewer.cmdInspectBrowser()"/>
   </broadcasterset>
   
   <!--============================= POPUPS ============================== -->
 
   <popupset id="psPopups">
     <popup id="ppDOMContext" oncreate="viewer.onContextCreate(this)">
-      <menuitem value="&cmdDeleteNode.label;" observes="cmdDeleteNode"/>
-      <menuitem value="&cmdInspectBrowser.label;" observes="cmdInspectBrowser"/>
+      <menuitem label="&cmdDeleteNode.label;" observes="cmdDeleteNode"/>
+      <menuitem label="&cmdInspectBrowser.label;" observes="cmdInspectBrowser"/>
     </popup>
   </popupset>
 
   <!-- ::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
     The template for this tree will be filled in by inTreeTableBuilder
     ::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::: -->
   
   <tree id="trDOMTree" 
--- a/resources/content/viewers/dom/findDialog.xul
+++ b/resources/content/viewers/dom/findDialog.xul
@@ -38,30 +38,30 @@
         <row id="rwRow2">
           <text id="txText2" value=""/>
           <textbox id="tfText2" observes="cmdFind"/>
         </row>
       </rows>
     </grid>
     <box orient="horizontal" autostretch="never" style="vertical-align: top">
       <titledbox id="tbxType">
-        <title value="Search By"/>
+        <label value="Search By"/>
         <radiogroup id="rgType" orient="vertical">
-          <radio id="rdType_id" value="ID" oncommand="dialog.toggleType('id')"/>
-          <radio id="rdType_tag" value="Tag" oncommand="dialog.toggleType('tag')"/>
-          <radio id="rdType_attr" value="Attribute" oncommand="dialog.toggleType('attr')"/>
+          <radio id="rdType_id" label="ID" oncommand="dialog.toggleType('id')"/>
+          <radio id="rdType_tag" label="Tag" oncommand="dialog.toggleType('tag')"/>
+          <radio id="rdType_attr" label="Attribute" oncommand="dialog.toggleType('attr')"/>
         </radiogroup>
       </titledbox>
       <titledbox id="tbxDirection" hide="true">
-        <title value="Direction"/>
+        <label value="Direction"/>
         <radiogroup id="rgDirection" orient="horizontal">
-          <radio id="rdDir_up" value="Up"/>
-          <radio id="rdDir_down" value="Down"/>
+          <radio id="rdDir_up" label="Up"/>
+          <radio id="rdDir_down" label="Down"/>
         </radiogroup>
       </titledbox>
     </box>
   </box>
 
   <box orient="vertical">
-    <button value="Find" observes="cmdFind" default="true"/>
-    <button value="Close" observes="cmdClose"/>
+    <button label="Find" observes="cmdFind" default="true"/>
+    <button label="Close" observes="cmdClose"/>
   </box>
 </window>
--- a/resources/content/viewers/dom/popupOverlay.xul
+++ b/resources/content/viewers/dom/popupOverlay.xul
@@ -3,70 +3,70 @@
 <!DOCTYPE rdf SYSTEM "chrome://inspector/locale/viewers/dom.dtd">
 
 <overlay id="ovDOMViewerPopup"
         xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
   
   <popupset id="ppsGlobalPopupset">
     <menupopup id="ppViewerContext-dom">
       <menuitem id="item:find"
-        value="&cmdShowFindDialog.label;"
+        label="&cmdShowFindDialog.label;"
         accesskey="&cmdShowFindDialog.accesskey;"
         key="key:find"
         observes="cmd:find"/>
 
       <menuitem id="item:findNext"
-        value="&cmdFindNext.label;"
+        label="&cmdFindNext.label;"
         key="key:findNext"
         observes="cmd:findNext"/>
 
       <menuseparator/>
 
       <menuitem id="item:toggleAnon"
         class="menuitem-iconic"
         type="checkbox"
-        value="&cmdToggleAnonContent.label;"
+        label="&cmdToggleAnonContent.label;"
         accesskey="&cmdToggleAnonContent.accesskey;"
         observes="cmd:toggleAnon"/>
 
       <menuseparator/>
 
       <menuitem id="item:showColumns"
-        value="&cmdShowColumnsDialog.label;"
+        label="&cmdShowColumnsDialog.label;"
         accesskey="&cmdShowColumnsDialog.accesskey;"
         observes="cmd:showColumns"/>
 
     </menupopup>
   </popupset>
 
   <menupopup id="mppSearch">
 
     <menuitem id="item:find" position="1"
-      value="&cmdShowFindDialog.label;"
+      label="&cmdShowFindDialog.label;"
       accesskey="&cmdShowFindDialog.accesskey;"
       key="key:find"
       observes="cmd:find"/>
 
     <menuitem id="item:findNext" position="2"
-      value="&cmdFindNext.label;"
+      label="&cmdFindNext.label;"
       key="key:findNext"
       observes="cmd:findNext"/>
     
     <menuseparator position="3"/>
 
     <menuitem id="item:selectByClick" position="4"
-      value="&cmdSelectByClick.label;"
+      label="&cmdSelectByClick.label;"
       accesskey="&cmdSelectByClick.accesskey;"
       observes="cmd:selectByClick"/>
   </menupopup>
 
   <menupopup id="mppView">
     <menuitem id="item:toggleAnon"
       class="menuitem-iconic"
       type="checkbox"
-      value="&cmdToggleAnonContent.label;"
+      label="&cmdToggleAnonContent.label;"
       accesskey="&cmdToggleAnonContent.accesskey;"
       observes="cmd:toggleAnon"/>
   </menupopup>
 
 </overlay>
 
         
\ No newline at end of file
--- a/resources/content/viewers/nodeElement/nodeElement.xul
+++ b/resources/content/viewers/nodeElement/nodeElement.xul
@@ -20,19 +20,19 @@
   <script type="application/x-javascript" src="chrome://inspector/content/utils.js"/>
   <script type="application/x-javascript" src="chrome://inspector/content/jsutil/xpcom/XPCU.js"/>
   <script type="application/x-javascript" src="chrome://inspector/content/jsutil/rdf/RDFU.js"/>
 
   <!--============================= POPUPS ============================== -->
 
   <popupset id="psPopups">
     <popup id="ppAttrContext">
-      <menuitem value="New Attribute" oncommand="viewer.cmdNewAttribute()"/>
-      <menuitem value="Edit" oncommand="viewer.cmdEditSelectedAttribute()"/>
-      <menuitem value="Delete" oncommand="viewer.cmdDeleteSelectedAttribute()"/>
+      <menuitem label="New Attribute" oncommand="viewer.cmdNewAttribute()"/>
+      <menuitem label="Edit" oncommand="viewer.cmdEditSelectedAttribute()"/>
+      <menuitem label="Delete" oncommand="viewer.cmdDeleteSelectedAttribute()"/>
     </popup>
   </popupset>
 
   <!--============================= CONTENT ============================== -->
 
   <grid>
     <columns>
       <column/>
@@ -66,18 +66,18 @@
     <treecolgroup>
       <treecol id="tcolNodeName" flex="1"/>
       <splitter class="tree-splitter"/>
       <treecol id="tcolNodeValue" flex="1"/>
     </treecolgroup>
     
     <treehead>
       <treerow>
-        <treecell class="treecell-header" value="Name"/>
-        <treecell class="treecell-header" value="Value"/>
+        <treecell class="treecell-header" label="Name"/>
+        <treecell class="treecell-header" label="Value"/>
       </treerow>
     </treehead>
 
     <treechildren flex="1"/>
 
     <template>
       <rule>
         <conditions>
@@ -89,18 +89,18 @@
           <binding subject="?node" predicate="&INS-rdf;nodeName" object="?nodeName"/>
           <binding subject="?node" predicate="&INS-rdf;nodeValue" object="?nodeValue"/>
         </bindings>
 
         <action>
           <treechildren>
             <treeitem uri="?node">
               <treerow>
-                <treecell value="?nodeName" ins-type="name"/>
-                <treecell value="?nodeValue" class="treecell-editable" ins-type="value"/>
+                <treecell label="?nodeName" ins-type="name"/>
+                <treecell label="?nodeValue" class="treecell-editable" ins-type="value"/>
               </treerow>
             </treeitem>
           </treechildren>
         </action>
       </rule>
     </template>
 
   </tree>
--- a/resources/content/viewers/styleRules/styleRules.xul
+++ b/resources/content/viewers/styleRules/styleRules.xul
@@ -20,29 +20,29 @@
   <script type="application/x-javascript" src="chrome://inspector/content/utils.js"/>
   <script type="application/x-javascript" src="chrome://inspector/content/jsutil/xpcom/XPCU.js"/>
   <script type="application/x-javascript" src="chrome://inspector/content/jsutil/rdf/RDFU.js"/>
 
   <!--============================= POPUPS ============================== -->
 
   <popupset id="psPopups">
     <popup id="ppStyleRuleContext" oncreate="return viewer.onCreateRulePopup()">
-      <menuitem value="New" oncommand="viewer.cmdNewRule()"/>
-      <menuitem value="Delete" oncommand="viewer.cmdDeleteSelectedRule()"/>
-      <menuitem value="Disable" oncommand="viewer.cmdToggleSelectedRule()"/>
+      <menuitem label="New" oncommand="viewer.cmdNewRule()"/>
+      <menuitem label="Delete" oncommand="viewer.cmdDeleteSelectedRule()"/>
+      <menuitem label="Disable" oncommand="viewer.cmdToggleSelectedRule()"/>
       <menuseparator/>
-      <menuitem value="Open File in Editor" oncommand="viewer.cmdOpenSelectedFileInEditor()"/>
+      <menuitem label="Open File in Editor" oncommand="viewer.cmdOpenSelectedFileInEditor()"/>
     </popup>
 
     <popup id="ppStyleDecContext">
-      <menuitem value="New Property" oncommand="viewer.cmdNewProperty()"/>
-      <menuitem value="Edit" oncommand="viewer.cmdEditSelectedProperty()"/>
-      <menuitem value="Delete" oncommand="viewer.cmdDeleteSelectedProperty()"/>
+      <menuitem label="New Property" oncommand="viewer.cmdNewProperty()"/>
+      <menuitem label="Edit" oncommand="viewer.cmdEditSelectedProperty()"/>
+      <menuitem label="Delete" oncommand="viewer.cmdDeleteSelectedProperty()"/>
       <menuseparator/>
-      <menuitem value="Toggle !important" oncommand="viewer.cmdToggleSelectedImportant()"/>
+      <menuitem label="Toggle !important" oncommand="viewer.cmdToggleSelectedImportant()"/>
     </popup>
   </popupset>
 
   <!--============================= CONTENT ============================== -->
 
   <box orient="vertical">
     <tree id="trStyleRules"
       flex="1" context="ppStyleRuleContext"
@@ -62,20 +62,20 @@
             <binding subject="?rule" predicate="&INS-rdf;Weight" object="?weight"/>
             <binding subject="?rule" predicate="&INS-rdf;LineNum" object="?lineNum"/>
           </bindings>
 
           <action>
             <treechildren>
               <treeitem uri="?rule">
                 <treerow>
-                  <treecell value="?selector"/>
-                  <treecell value="?fileURL"/>
-                  <treecell value="?weight"/>
-                  <treecell value="?lineNum"/>
+                  <treecell label="?selector"/>
+                  <treecell label="?fileURL"/>
+                  <treecell label="?weight"/>
+                  <treecell label="?lineNum"/>
                 </treerow>
               </treeitem>
             </treechildren>
           </action>
         </rule>
       </template>
 
       <treecolgroup>
@@ -85,20 +85,20 @@
         <splitter class="tree-splitter"/>
         <treecol id="tcolNodeValue" flex="1"/>
         <splitter class="tree-splitter"/>
         <treecol id="tcolNodeValue" flex="1"/>
       </treecolgroup>
       
       <treehead>
         <treerow>
-          <treecell class="treecell-header" value="Selector"/>
-          <treecell class="treecell-header" value="File"/>
-          <treecell class="treecell-header" value="Weight"/>
-          <treecell class="treecell-header" value="Line"/>
+          <treecell class="treecell-header" label="Selector"/>
+          <treecell class="treecell-header" label="File"/>
+          <treecell class="treecell-header" label="Weight"/>
+          <treecell class="treecell-header" label="Line"/>
         </treerow>
       </treehead>
 
       <treechildren flex="1"/>
     </tree>
   </box>
 
   <splitter orient="horizontal" collapse="after">
@@ -123,36 +123,36 @@
           <binding subject="?prop" predicate="&INS-rdf;PropertyValue" object="?value"/>
         </bindings>
 
         <action>
           <treechildren>
             <treeitem uri="?prop">
               <treerow>
                 <treecell class="treecell-priority treecell-iconic" ins-type="priority" ins:priority="?priority"/>
-                <treecell value="?name" ins-type="name"/>
-                <treecell class="treecell-editable" value="?value" ins-type="value"/>
+                <treecell label="?name" ins-type="name"/>
+                <treecell class="treecell-editable" label="?value" ins-type="value"/>
               </treerow>
             </treeitem>
           </treechildren>
         </action>
       </rule>
     </template>
 
     <treecolgroup>
       <treecol id="tcolPropPriority"/>
       <treecol id="tcolPropName" flex="1"/>
       <splitter class="tree-splitter"/>
       <treecol id="tcolPropValue" flex="1"/>
     </treecolgroup>
     
     <treehead>
       <treerow>
-        <treecell class="treecell-header" value="!"/>
-        <treecell class="treecell-header" value="Name"/>
-        <treecell class="treecell-header" value="Value"/>
+        <treecell class="treecell-header" label="!"/>
+        <treecell class="treecell-header" label="Name"/>
+        <treecell class="treecell-header" label="Value"/>
       </treerow>
     </treehead>
 
     <treechildren flex="1"/>
   </tree>    
 
 </window>