Bug 389689 - Have the dialog for nsIContentDispatchChooser use the width property. r=mano, ui-r=beltzner

toolkit/mozapps/handling/content/dialog.js

@@ -210,10 +210,10 @@ var dialog = {
   */
   onCheck: function onCheck()
   {
-    document.getElementById("remember-text").hidden =
-      !document.getElementById("remember").checked;
-
-    window.sizeToContent();
+    if (document.getElementById("remember").checked)
+      document.getElementById("remember-text").setAttribute("visible", "true");
+    else
+      document.getElementById("remember-text").removeAttribute("visible");
   },
 
   /////////////////////////////////////////////////////////////////////////////

toolkit/mozapps/handling/content/dialog.xul

@@ -45,7 +45,8 @@
 <dialog id="handling"
         ondialogaccept="return dialog.onAccept();"
         onload="dialog.initialize();"
-        style="width: 320px;"
+        width="&window.width;" height="&window.height;"
+        persist="width height screenX screenY sizemode"
         xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 
   <script src="chrome://mozapps/content/handling/dialog.js" type="application/javascript"/>
@@ -73,6 +74,6 @@
   </vbox>
 
   <checkbox id="remember" oncommand="dialog.onCheck();"/>
-  <description id="remember-text" hidden="true"/>
+  <description id="remember-text"/>
 
 </dialog>

toolkit/mozapps/handling/content/handler.css

@@ -1,3 +1,7 @@
 richlistitem[type="handler"] {
   -moz-binding: url('chrome://mozapps/content/handling/handler.xml#handler');
 }
+
+#remember-text:not([visible]) {
+  visibility: hidden;
+}

toolkit/mozapps/handling/content/handling.dtd

@@ -1,3 +1,5 @@
+<!ENTITY window.width "320">
+<!ENTITY window.height "250">
 <!ENTITY ChooseApp.description "Choose an Application">
 <!ENTITY ChooseApp.label "Choose...">
 <!ENTITY ChooseApp.accessKey "C">