Update Confirmation UI for saved password change

chrome/browser/ui/passwords/manage_passwords_ui_controller.h

Index: chrome/browser/ui/passwords/manage_passwords_ui_controller.h
diff --git a/chrome/browser/ui/passwords/manage_passwords_ui_controller.h b/chrome/browser/ui/passwords/manage_passwords_ui_controller.h
index 82d3bb6c36d461cff6ada49c9baab83535fdf42d..05a69007c37108d44d9942cc4ac9aceef202f1c8 100644
--- a/chrome/browser/ui/passwords/manage_passwords_ui_controller.h
+++ b/chrome/browser/ui/passwords/manage_passwords_ui_controller.h
@@ -76,10 +76,15 @@ class ManagePasswordsUIController
       const password_manager::PasswordStoreChangeList& changes) override;
 
   // Called from the model when the user chooses to save a password; passes the
-  // action off to the FormManager. The controller MUST be in a pending state,
-  // and WILL be in MANAGE_STATE after this method executes.
+  // action to the FormManager. The controller MUST be in a pending state, and

[vabr (Chromium), 2015/07/24 08:24:29]

nit: "must" and "will". RFCs and other standards use the upper-case to emphasise
that those terms have a specially defined meaning for the standards, but I don't
think there was any such definition for the comments here.

[vabr (Chromium), 2015/07/24 08:24:29]

nit: PasswordFormManager or "form manager". Just FormManager looks like a class
name, but there is no such class.

[dvadym, 2015/07/24 16:42:39]

Done.

[dvadym, 2015/07/24 16:42:39]

Done.

+  // WILL be in MANAGE_STATE after this method executes.
   virtual void SavePassword();
 
+  // Called from the model when the user chooses to update a password; passes
+  // the action to the FormManager. The controller MUST be in a pending state,
+  // and WILL be in MANAGE_STATE after this method executes.
+  virtual void UpdatePassword(const autofill::PasswordForm& password_form);
+
   // Called from the model when the user chooses a credential.
   // The controller MUST be in a pending credentials state.
   virtual void ChooseCredential(
@@ -151,6 +156,8 @@ class ManagePasswordsUIController
   // The pieces of saving and blacklisting passwords that interact with
   // FormManager, split off into internal functions for testing/mocking.
   virtual void SavePasswordInternal();
+  virtual void UpdatePasswordInternal(
+      const autofill::PasswordForm& password_form);
   virtual void NeverSavePasswordInternal();
 
   // Called when a PasswordForm is autofilled, when a new PasswordForm is