Bug 143663 Helper Apps listbox scrolls badly r=varga sr=alecf a=asa

xpfe/components/prefwindow/resources/content/pref-applications-new.js

@@ -176,7 +176,6 @@ function onOK()
       window.arguments[0].updateSelf = true;
   }
 
-  window.opener.gNewTypeRV = gMIMEField.value;
   window.close();  
   return false;
 }

xpfe/components/prefwindow/resources/content/pref-applications.js

@@ -18,7 +18,6 @@
  * Contributor(s):
  */
 
-var gNewTypeRV    = null;
 var gUpdateTypeRV = null;
 var gList   = null;
 var gDS     = null;
@@ -35,16 +34,12 @@ function newType()
 {
   var handlerOverride = new HandlerOverride();
   window.openDialog("chrome://communicator/content/pref/pref-applications-edit.xul", "appEdit", "chrome,modal=yes,resizable=no", handlerOverride);
-  if (gNewTypeRV) {
-    gList.builder.rebuild();
-    gNewTypeRV = null;
-  }
 }
 
 function removeType()
 {
   // Only prompt if setting is "useHelperApp".
-  var uri = gList.selectedItems[0].id;
+  var uri = gList.view.getResourceAtIndex(gList.currentIndex).Value;
   var handlerOverride = new HandlerOverride(uri);
   if ( !handlerOverride.useSystemDefault && !handlerOverride.saveToDisk ) {
     var titleMsg = gPrefApplicationsBundle.getString("removeHandlerTitle");
@@ -57,14 +52,13 @@ function removeType()
     }
   }
   removeOverride(handlerOverride.mimeType);
-  gList.builder.rebuild();
   selectApplication();
 }
 
 function editType()
 {
-  if (gList.selectedItems && gList.selectedItems[0]) {
-    var uri = gList.selectedItems[0].id;
+  if (gList.currentIndex >= 0) {
+    var uri = gList.view.getResourceAtIndex(gList.currentIndex).Value;
     var handlerOverride = new HandlerOverride(uri);
     window.openDialog("chrome://communicator/content/pref/pref-applications-edit.xul", "appEdit", "chrome,modal=yes,resizable=no", handlerOverride);
     selectApplication();
@@ -197,8 +191,8 @@ function Startup()
 
 function selectApplication()
 {
-  if (gList.selectedItems && gList.selectedItems.length && gList.selectedItems[0]) {
-    var uri = gList.selectedItems[0].id;
+  if (gList.currentIndex >= 0) {
+    var uri = gList.view.getResourceAtIndex(gList.currentIndex).Value;
     var handlerOverride = new HandlerOverride(uri);
     gExtensionField.setAttribute("value", handlerOverride.extensions);
     gMIMEDescField.setAttribute("value", handlerOverride.description);

xpfe/components/prefwindow/resources/content/pref-applications.xul

@@ -47,11 +47,14 @@
     <label value="&fileTypes.label;"/>
     <vbox flex="1">
       <hbox flex="1">
-        <listbox id="appList" flex="1"
-              sortResource="http://home.netscape.com/NC-rdf#value"
-              sortDirection="ascending"
+        <tree id="appList" flex="1" seltype="single" hidecolumnpicker="true"
               datasources="rdf:null" ref="urn:mimetypes"
+              flags="dont-build-content"
               onselect="selectApplication();">
+          <treecols>
+            <treecol sort="?value" sortActive="true" sortDirection="ascending"
+                     flex="1" hideheader="true"/>
+          </treecols>
           <template>
             <rule>
               <conditions>
@@ -68,11 +71,17 @@
                         predicate="http://home.netscape.com/NC-rdf#editable"/>
               </conditions>
               <action> 
-                <listitem uri="?type" label="?value"/>
+                <treechildren>
+                  <treeitem uri="?type">
+                    <treerow>
+                      <treecell label="?value"/>
+                    </treerow>
+                  </treeitem>
+                </treechildren>
               </action>
             </rule>        
           </template>
-        </listbox>
+        </tree>
         <vbox>
           <button id="newTypeButton" 
                   label="&newTypeButton.label;" accesskey="&newTypeButton.accesskey;"