From 68b9c57d27ebda53d3fb228a2b7163dfe0124cc6 Mon Sep 17 00:00:00 2001
From: "bryner%brianryner.com"
 <bryner%brianryner.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Mon, 1 Sep 2003 21:11:30 +0000
Subject: [PATCH] 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.

git-svn-id: svn://10.0.0.236/trunk@146499 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../components/passwordmgr/base/nsPasswordManager.cpp       | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/mozilla/toolkit/components/passwordmgr/base/nsPasswordManager.cpp b/mozilla/toolkit/components/passwordmgr/base/nsPasswordManager.cpp
index 3667f8924ca..b0a06e4c161 100644
--- a/mozilla/toolkit/components/passwordmgr/base/nsPasswordManager.cpp
+++ b/mozilla/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;
       }