Fix for bug 199524. File bookmark... dialog no longer resizable

r=jag, sr=peterv

xpfe/components/bookmarks/resources/addBookmark.js

@@ -322,3 +322,14 @@ function toggleGroup()
   gFld_Name.focus();
   onFieldInput();
 }
+
+function persistTreeSize()
+{
+  if (!document.getElementById("folderbox").hidden) {
+    var bookmarkView = document.getElementById("bookmarks-view");
+    bookmarkView.setAttribute("height", bookmarkView.boxObject.height);
+    document.persist("bookmarks-view", "height");
+    bookmarkView.setAttribute("width", bookmarkView.boxObject.width);
+    document.persist("bookmarks-view", "width");
+  }
+}

xpfe/components/bookmarks/resources/addBookmark.xul

@@ -41,6 +41,7 @@
         ondialogaccept="return onOK(event)"
         title="&newBookmark.title;" selectFolderTitle="&selectFolder.title;"
         onload="Startup();"
+        onunload="persistTreeSize();"
         persist="screenX screenY"
         screenX="24" screenY="24">
    
@@ -94,7 +95,8 @@
   <vbox id="folderbox" flex="1">
     <label id="destinationLabel" value="&destination.label;"/>
     <bookmarks-tree id="bookmarks-view" flex="1" type="folders"
-                    rows="10" seltype="single"/>
+                    rows="10" seltype="single"
+                    persist="width height"/>
 
     <separator id="folderbuttonseparator" class="thin"/>
 

xpfe/components/bookmarks/resources/bookmarks.js

@@ -505,7 +505,7 @@ var BookmarksCommand = {
   {
     var rv = { selectedFolder: null };      
     openDialog("chrome://communicator/content/bookmarks/addBookmark.xul", "", 
-               "centerscreen,chrome,modal=yes,dialog=yes,resizable=no", null, null, null, null, "selectFolder", rv);
+               "centerscreen,chrome,modal=yes,dialog=yes,resizable=yes", null, null, null, null, "selectFolder", rv);
     if (!rv.target)
       return;
     
@@ -894,7 +894,7 @@ var BookmarksController = {
       var folder = aTarget.parent.Value;
       var rv = { newBookmark: null };
       openDialog("chrome://communicator/content/bookmarks/addBookmark.xul", "", 
-                 "centerscreen,chrome,modal=yes,dialog=yes,resizable=no", null, null, folder, null, "newBookmark", rv);
+                 "centerscreen,chrome,modal=yes,dialog=yes,resizable=yes", null, null, folder, null, "newBookmark", rv);
       break;
     case "cmd_bm_newseparator":
       BookmarksCommand.createNewSeparator(aTarget);
@@ -1506,7 +1506,7 @@ var BookmarksUtils = {
     }
 
     openDialog("chrome://communicator/content/bookmarks/addBookmark.xul", "",
-               "centerscreen,chrome,dialog=yes,resizable=no,dependent",
+               "centerscreen,chrome,dialog=yes,resizable=yes,dependent",
                currentTabInfo.name, currentTabInfo.url, null,
                currentTabInfo.charset, "addGroup" + (aSelect ? ",group" : ""), tabsInfo);
   },
@@ -1541,7 +1541,7 @@ var BookmarksUtils = {
 
     if (aShowDialog) {
       openDialog("chrome://communicator/content/bookmarks/addBookmark.xul", "",
-                 "centerscreen,chrome,dialog=yes,resizable=no,dependent", aTitle, aURL, null, aCharset);
+                 "centerscreen,chrome,dialog=yes,resizable=yes,dependent", aTitle, aURL, null, aCharset);
     }
     else {
       // User has elected to override the file dialog and always file bookmarks