Allow editing passwords in settings/passwords

chrome/browser/ui/passwords/password_manager_presenter.h

Index: chrome/browser/ui/passwords/password_manager_presenter.h
diff --git a/chrome/browser/ui/passwords/password_manager_presenter.h b/chrome/browser/ui/passwords/password_manager_presenter.h
index 66c68462967c25cc4689f18f2bad21fa34f7e568..f7c700a716cbaff73959e82453541f6a3ff3dd42 100644
--- a/chrome/browser/ui/passwords/password_manager_presenter.h
+++ b/chrome/browser/ui/passwords/password_manager_presenter.h
@@ -46,6 +46,18 @@ class PasswordManagerPresenter
   // Gets the password exception entry at |index|.
   const autofill::PasswordForm* GetPasswordException(size_t index);
 
+  // Checks if |origin| is valid for adding a new password entry.
+  static bool CheckOriginValidityForAdding(const GURL& origin);
+
+  // Adds a new password entry with |origin|, |username_value|, and
+  // |password_value|.
+  void AddPassword(const GURL& origin,

[vabr (Chromium), 2014/08/25 07:35:07]

Please document restrictions on |origin| and |password_value| (to match the
NOTREACHED() checks in the .cc file).

[jaekyeom, 2014/08/26 05:34:48]

Done.

+                   const base::string16& username_value,
+                   const base::string16& password_value);
+
+  // Updates the entry at |index| with |password_value|.
+  void UpdatePassword(size_t index, const base::string16& password_value);

[vabr (Chromium), 2014/08/25 07:35:07]

Please document restrictions on |password_value| (to match the NOTREACHED()
check in the .cc file).

[jaekyeom, 2014/08/26 05:34:48]

Done.

+
   // Removes the saved password entry at |index|.
   // |index| the entry index to be removed.
   void RemoveSavedPassword(size_t index);