fix bug 26305, wallet editor is empty, r=ben

extensions/wallet/editor/WalletEditor.js

@@ -40,17 +40,11 @@ var BREAK;
 
 /*** =================== ?? =================== ***/
 
-//?? Why do we need to call ViewEntries twice?  If we don't
-//?? then we don't get the display the first time an entry is selected.  Same
-//?? for ViewSynonyms.
-
 function ViewEntriesFromXul(){
   ViewEntries();
-  ViewEntries();
 }
 function ViewSynonymsFromXul(){
   ViewSynonyms();
-  ViewSynonyms();
 }
 
 /*** =================== STARTING AND STOPPING =================== ***/
@@ -66,8 +60,6 @@ function Startup()
     return; /* user failed to unlock the database */
   }
   ViewSchema(); /* create the display of schemas */
-  ViewSchema(); /* create the display of schemas */
-  //??why do we need to do above twice?  Without it, first schema is not displayed
   doSetOKCancel(onOK, null); /* register the onOK method */
   window.sizeToContent();
 }

extensions/wallet/editor/WalletEditor.xul

@@ -49,7 +49,8 @@
         <html:div style="width: 195px;">
           <box><html:div>&treehead.schemaname.label;</html:div></box>
           <tree id="schematree" class="inset" style="height: 250px; width: 195px;" align="vertical" 
-            onclick="ViewEntriesFromXul();">
+            onclick="ViewEntries();">
+            <treecol width="100%"/>
             <treechildren id="schemalist"/>
           </tree>
           <spring flex="5%"/>
@@ -63,13 +64,15 @@
         <html:div style="width: 195px;">
           <box><html:div>&treehead.entries.label;</html:div></box>
           <tree id="entrytree" class="inset" style="height: 113px; width: 195px;" align="vertical" 
-            onclick="ViewSynonymsFromXul();">
+            onclick="ViewSynonyms();">
+            <treecol width="100%"/>
             <treechildren id="entrieslist"/>
           </tree>
           <spring style="height: 7px;"/>
           <box><html:div>&treehead.synonyms.label;</html:div></box>
           <tree id="synonymtree" class="inset" style="height: 113px; width: 195px;" align="vertical" 
             onclick="SynonymSelected();">
+            <treecol width="100%"/>
             <treechildren id="synonymslist"/>
           </tree>
         </html:div>

extensions/wallet/signonviewer/SignonViewer.xul

@@ -83,6 +83,7 @@
           <tree id="ignoretree" class="inset" style="height: 280px; width: 440px;" align="vertical"
                 onclick="HandleEvent(event);"
                 onkeypress="HandleEvent(event,1)">
+            <treecol width="100%"/>
             <treehead>
               <treerow>
                 <treecell value="&treehead.site.label;"/>
@@ -105,6 +106,7 @@
           <tree id="nopreviewtree" class="inset" style="height: 280px; width: 440px;" align="vertical"
                 onclick="HandleEvent(event);"
                 onkeypress="HandleEvent(event,2)">
+            <treecol width="100%"/>
             <treehead>
               <treerow>
                 <treecell value="&treehead.site.label;"/>
@@ -127,6 +129,7 @@
           <tree id="nocapturetree" class="inset" style="height: 280px; width: 440px;" align="vertical"
                 onclick="HandleEvent(event);"
                 onkeypress="HandleEvent(event,3)">
+            <treecol width="100%"/>
             <treehead>
               <treerow>
                 <treecell value="&treehead.site.label;"/>