| Index: chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
|
| diff --git a/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc b/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
|
| index cba75401d0cda884cf6daacb29abf59e68e60791..bbc7c14eee211c763cc3dfb86ee12600ae46ff0d 100644
|
| --- a/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
|
| +++ b/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
|
| @@ -218,8 +218,11 @@ void ManagePasswordsUIController::ChooseCredential(
|
| void ManagePasswordsUIController::SavePasswordInternal() {
|
| password_manager::PasswordFormManager* form_manager =
|
| passwords_data_.form_manager();
|
| - DCHECK(form_manager);
|
| - form_manager->Save();
|
| + // TODO(vasilii): it's not OK to call SavePassword() when |form_manager| is 0.
|
| + // If this is a cause of http://crbug.com/468474 then we should hide the
|
| + // bubble when ManagePasswordsUIController changes its internal state.
|
| + if (form_manager)
|
| + form_manager->Save();
|
| }
|
|
|
| void ManagePasswordsUIController::NeverSavePassword() {
|
|
|