Bug 282183 sync xpfe listbox.xml with toolkit listbox.xml. (Bv1) nits, in the meantime. Patch by Serge GAUTHERIE <gautheri@noos.fr>, r/sr=neil

toolkit/content/widgets/listbox.xml

@@ -354,7 +354,7 @@
           } else {
             currentItem = startItem;
           }
-            
+
           while (currentItem) {
             if (currentItem.localName == "listitem")
               this.addItemToSelection(currentItem);
@@ -376,7 +376,7 @@
             this.removeItemFromSelection(currentItem);
 
           this._suppressOnSelect = suppressSelect;
-          
+
           this._fireOnSelect();
         ]]>
         </body>
@@ -527,7 +527,7 @@
               newIndex = 0;
             var numItems = this.getRowCount();
             if (newIndex > numItems - 1)
-              var newIndex = numItems - 1;
+              newIndex = numItems - 1;
             var newItem = this.getItemAtIndex(newIndex);
             if (newItem) {
               this.ensureIndexIsVisible(newIndex);

xpfe/global/resources/content/bindings/listbox.xml

@@ -577,14 +577,14 @@
       <handler event="keypress" keycode="vk_page_down" modifiers="control shift any"
                phase="target" action="moveByOffset(this.scrollOnePage(1), !event.ctrlKey, event.shiftKey);"/>
       <handler event="keypress" key=" " phase="target">
-      <![CDATA[
-        if (this.currentItem) {
-          if (this.currentItem.getAttribute("type") != "checkbox")
-            this.addItemToSelection(this.currentItem);
-          else if (!this.currentItem.disabled)
-            this.currentItem.checked = !this.currentItem.checked;
-        }
-      ]]>
+        <![CDATA[
+          if (this.currentItem) {
+            if (this.currentItem.getAttribute("type") != "checkbox")
+              this.addItemToSelection(this.currentItem);
+            else if (!this.currentItem.disabled)
+              this.currentItem.checked = !this.currentItem.checked;
+          }
+        ]]>
       </handler>
       <handler event="keypress" key=" " modifiers="control" phase="target">
         <![CDATA[