bug 217954/216395 - we need to assign to |userField| regardless of whether there is a value, so we don't crash later when trying to attach autocomplete to the field.

toolkit/components/passwordmgr/base/nsPasswordManager.cpp

@@ -636,8 +636,7 @@ nsPasswordManager::OnStateChange(nsIWebProgress* aWebProgress,
 
       if (temp) {
         temp->GetValue(oldUserValue);
-        if (oldUserValue.IsEmpty())
-          userField = temp;
+        userField = temp;
       } else {
         continue;
       }
@@ -649,8 +648,7 @@ nsPasswordManager::OnStateChange(nsIWebProgress* aWebProgress,
 
       if (temp) {
         temp->GetValue(oldPassValue);
-        if (oldPassValue.IsEmpty())
-          passField = temp;
+        passField = temp;
       } else {
         continue;
       }