Follow up to bug 370306 - Disable background-autocomplete on paste for now as it is not working, bug 456550 will fix it again later. r=bienvenu,sr=Neil
authorMark Banner <bugzilla@standard8.plus.com>
Tue, 23 Sep 2008 22:56:58 +0100
changeset 408 afc9faafb312235189ba0e52b7af541bcd4b26b1
parent 407 f010bf7956d6791afb9fa4e50eedb4d2ce6ad0e2
child 409 dba6f5a634ec4811889485b1788928057c89773d
push idunknown
push userunknown
push dateunknown
reviewersbienvenu, Neil
bugs370306, 456550
Follow up to bug 370306 - Disable background-autocomplete on paste for now as it is not working, bug 456550 will fix it again later. r=bienvenu,sr=Neil
mail/components/compose/content/addressingWidgetOverlay.js
mailnews/compose/resources/content/addressingWidgetOverlay.js
--- a/mail/components/compose/content/addressingWidgetOverlay.js
+++ b/mail/components/compose/content/addressingWidgetOverlay.js
@@ -1133,16 +1133,18 @@ AutomatedAutoCompleteHandler.prototype =
   autoCompleteNextAddress:function()
   {
     this.numSessionsToSearch = 0;
     this.numSessionsSearched = 0;
     this.searchResults = new Array;
 
     if (this.indexIntoNames < this.numNamesToComplete && this.namesToComplete[this.indexIntoNames])
     {
+      /* XXX This is used to work, until switching to the new toolkit broke it
+         We should fix it see bug 456550.
       if (this.namesToComplete[this.indexIntoNames].search('@') == -1) // don't autocomplete if address has an @ sign in it
       {
         // make sure total session count is updated before we kick off ANY actual searches
         if (gAutocompleteSession)
           this.numSessionsToSearch++;
 
         if (gLDAPSession && gCurrentAutocompleteDirectory)
           this.numSessionsToSearch++;
@@ -1160,16 +1162,17 @@ AutomatedAutoCompleteHandler.prototype =
              this.processAllResults();
              return;
            }
         }
 
         if (gLDAPSession && gCurrentAutocompleteDirectory)
           gLDAPSession.onStartLookup(this.namesToComplete[this.indexIntoNames], null, this);
       }
+      */
 
       if (!this.numSessionsToSearch)
         this.processAllResults(); // ldap and ab are turned off, so leave text alone
     }
   },
 
   onStatus:function(aStatus)
   {
--- a/mailnews/compose/resources/content/addressingWidgetOverlay.js
+++ b/mailnews/compose/resources/content/addressingWidgetOverlay.js
@@ -1132,16 +1132,18 @@ AutomatedAutoCompleteHandler.prototype =
   autoCompleteNextAddress:function()
   {
     this.numSessionsToSearch = 0;
     this.numSessionsSearched = 0;
     this.searchResults = new Array;
 
     if (this.indexIntoNames < this.numNamesToComplete && this.namesToComplete[this.indexIntoNames])
     {
+      /* XXX This is used to work, until switching to the new toolkit broke it
+         We should fix it see bug 456550.
       if (this.namesToComplete[this.indexIntoNames].search('@') == -1) // don't autocomplete if address has an @ sign in it
       {
         // make sure total session count is updated before we kick off ANY actual searches
         if (gAutocompleteSession) 
           this.numSessionsToSearch++;
 
         if (gLDAPSession && gCurrentAutocompleteDirectory)
           this.numSessionsToSearch++;
@@ -1159,16 +1161,17 @@ AutomatedAutoCompleteHandler.prototype =
              this.processAllResults();
              return;
            }
         }
 
         if (gLDAPSession && gCurrentAutocompleteDirectory)
           gLDAPSession.onStartLookup(this.namesToComplete[this.indexIntoNames], null, this);
       }
+      */
 
       if (!this.numSessionsToSearch)
         this.processAllResults(); // ldap and ab are turned off, so leave text alone
     }
   },
 
   onStatus:function(aStatus) 
   {