Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7058)

Unified Diff: chrome/browser/ui/passwords/manage_passwords_ui_controller.cc

Issue 1151373006: Update Confirmation UI for saved password change (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clean up Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 b5c95902d2820cc498950a2bf2b6f897203868b2..af9da3d80022dbb3a56fb8e1271a081a4ebdec60 100644
--- a/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
+++ b/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
@@ -202,6 +202,14 @@ void ManagePasswordsUIController::SavePassword() {
UpdateBubbleAndIconVisibility();
}
+void ManagePasswordsUIController::UpdatePassword(
+ const autofill::PasswordForm& password_form) {
+ DCHECK(PasswordPendingUserDecision());
+ UpdatePasswordInternal(password_form);
+ passwords_data_.TransitionToState(password_manager::ui::MANAGE_STATE);
+ UpdateBubbleAndIconVisibility();
+}
+
void ManagePasswordsUIController::ChooseCredential(
const autofill::PasswordForm& form,
password_manager::CredentialType credential_type) {
@@ -249,6 +257,14 @@ void ManagePasswordsUIController::SavePasswordInternal() {
form_manager->Save();
}
+void ManagePasswordsUIController::UpdatePasswordInternal(
+ const autofill::PasswordForm& password_form) {
+ password_manager::PasswordFormManager* form_manager =
+ passwords_data_.form_manager();
+ DCHECK(form_manager);
+ form_manager->Update(password_form);
+}
+
void ManagePasswordsUIController::NeverSavePassword() {
DCHECK(PasswordPendingUserDecision());
NeverSavePasswordInternal();

Powered by Google App Engine
This is Rietveld 408576698