landing neil's fix for bug #193416

mailnews start page gets cleared when I open the pref window
regression caused by bandaid for bug #90337
r=jan, sr=jag.


git-svn-id: svn://10.0.0.236/trunk@140098 18797224-902f-48f8-a5cc-f745e15eee43

mozilla/mailnews/addrbook/resources/content/abMailListDialog.js

@@ -444,10 +444,6 @@ function awAppendNewRow(setFocus)
     if ( input && input.length == 1 )
     {
       input[0].setAttribute("value", "");
-      // XXX TODO
-      // band-aid for bug #190153 which was caused by a
-      // band-aid for #90337
-      input[0].value = "";
       input[0].setAttribute("id", "addressCol1#" + top.MAX_RECIPIENTS);
       
       if (input[0].getAttribute('focused') != '')

mozilla/mailnews/compose/resources/content/addressingWidgetOverlay.js

@@ -542,10 +542,6 @@ function awAppendNewRow(setFocus)
     if ( input && input.length == 1 )
     {
       input[0].setAttribute("value", "");
-      // XXX TODO
-      // band-aid for bug #190153 which was caused by a
-      // band-aid for #90337
-      input[0].value = "";
       input[0].setAttribute("id", "addressCol2#" + top.MAX_RECIPIENTS);
     
       //this copies the autocomplete sessions list from recipient#1 

mozilla/xpfe/components/autocomplete/resources/content/autocomplete.xml

@@ -40,7 +40,8 @@
 
       <constructor><![CDATA[
         // XXX bug 90337 band-aid until we figure out what's going on here
-        this.mInputElt.value = this.value;
+        if (this.value != this.mInputElt.value)
+          this.mInputElt.value = this.value;
         delete this.value;
 
         // set default property values