| Index: chrome/browser/password_manager/password_form_manager.cc
|
| diff --git a/chrome/browser/password_manager/password_form_manager.cc b/chrome/browser/password_manager/password_form_manager.cc
|
| index a75b3c8a987633e072a9da020d04b797cd936ed7..fc3ddc232537ae6f625a1c547124cddb9729fc12 100644
|
| --- a/chrome/browser/password_manager/password_form_manager.cc
|
| +++ b/chrome/browser/password_manager/password_form_manager.cc
|
| @@ -40,8 +40,7 @@ PasswordFormManager::PasswordFormManager(Profile* profile,
|
| manager_action_(kManagerActionNone),
|
| user_action_(kUserActionNone),
|
| submit_result_(kSubmitResultNotSubmitted),
|
| - should_save_password_(false),
|
| - should_blacklist_password_(false) {
|
| + password_action_(DO_NOTHING) {
|
| DCHECK(profile_);
|
| if (observed_form_.origin.is_valid())
|
| base::SplitString(observed_form_.origin.path(), '/', &form_path_tokens_);
|
| @@ -54,8 +53,7 @@ PasswordFormManager::~PasswordFormManager() {
|
| 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();
|
| + ApplyChange();
|
| }
|
|
|
| int PasswordFormManager::GetActionsTaken() {
|
| @@ -113,23 +111,11 @@ bool PasswordFormManager::DoesManage(const PasswordForm& form,
|
| }
|
|
|
| void PasswordFormManager::ApplyChange() {
|
| - DCHECK(!should_blacklist_password_ || !should_save_password_);
|
| - if (should_save_password_)
|
| + if (password_action_ == SAVE)
|
| Save();
|
| - else if (should_blacklist_password_)
|
| + else if (password_action_ == BLACKLIST)
|
| 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;
|
| + password_action_ = DO_NOTHING;
|
| }
|
|
|
| bool PasswordFormManager::IsBlacklisted() {
|
|
|