Allow editing passwords in settings/passwords

chrome/browser/ui/passwords/password_manager_presenter.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_PASSWORDS_PASSWORD_MANAGER_PRESENTER_H_
 #define CHROME_BROWSER_UI_PASSWORDS_PASSWORD_MANAGER_PRESENTER_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 28 matching lines @@
   void UpdatePasswordLists();
 
   void Initialize();
 
   // Gets the password entry at |index|.
   const autofill::PasswordForm* GetPassword(size_t index);
 
   // 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);
 
   // Removes the saved password exception entry at |index|.
   // |index| the entry index to be removed.
   void RemovePasswordException(size_t index);
 
   // Requests the plain text password for entry at |index| to be revealed.
   // |index| The index of the entry.
@@ skipping 70 matching lines @@
   // Used to determine whether or not to reveal plaintext passwords.
   base::TimeTicks last_authentication_time_;
 
   // UI view that owns this presenter.
   PasswordUIView* password_view_;
 
   DISALLOW_COPY_AND_ASSIGN(PasswordManagerPresenter);
 };
 
 #endif  // CHROME_BROWSER_UI_PASSWORDS_PASSWORD_MANAGER_PRESENTER_H_