Index: trunk/src/chrome/browser/password_manager/password_form_manager.cc |
=================================================================== |
--- trunk/src/chrome/browser/password_manager/password_form_manager.cc (revision 221694) |
+++ trunk/src/chrome/browser/password_manager/password_form_manager.cc (working copy) |
@@ -39,9 +39,7 @@ |
web_contents_(web_contents), |
manager_action_(kManagerActionNone), |
user_action_(kUserActionNone), |
- submit_result_(kSubmitResultNotSubmitted), |
- should_save_password_(false), |
- should_blacklist_password_(false) { |
+ submit_result_(kSubmitResultNotSubmitted) { |
DCHECK(profile_); |
if (observed_form_.origin.is_valid()) |
base::SplitString(observed_form_.origin.path(), '/', &form_path_tokens_); |
@@ -52,10 +50,6 @@ |
UMA_HISTOGRAM_ENUMERATION("PasswordManager.ActionsTaken", |
GetActionsTaken(), |
kMaxNumActionsTaken); |
- // In case the tab is closed before the next navigation occurs this will |
- // apply outstanding changes. |
- if (should_save_password_ || should_blacklist_password_) |
- ApplyChange(); |
} |
int PasswordFormManager::GetActionsTaken() { |
@@ -112,26 +106,6 @@ |
return true; |
} |
-void PasswordFormManager::ApplyChange() { |
- DCHECK(!should_blacklist_password_ || !should_save_password_); |
- if (should_save_password_) |
- Save(); |
- else if (should_blacklist_password_) |
- PermanentlyBlacklist(); |
- should_blacklist_password_ = false; |
- should_save_password_ = false; |
-} |
- |
-void PasswordFormManager::SavePassword() { |
- should_blacklist_password_ = false; |
- should_save_password_ = true; |
-} |
- |
-void PasswordFormManager::BlacklistPassword() { |
- should_save_password_ = false; |
- should_blacklist_password_ = true; |
-} |
- |
bool PasswordFormManager::IsBlacklisted() { |
DCHECK_EQ(state_, POST_MATCHING_PHASE); |
if (preferred_match_ && preferred_match_->blacklisted_by_user) |