Allow deleting autofill password suggestions on Shift+Delete

components/password_manager/core/browser/password_form_manager.h

Index: components/password_manager/core/browser/password_form_manager.h
diff --git a/components/password_manager/core/browser/password_form_manager.h b/components/password_manager/core/browser/password_form_manager.h
index a1d059589492736eed0e34e955bde29ce2f15c29..24effab78b0121356804c7a0cc520970771e246b 100644
--- a/components/password_manager/core/browser/password_form_manager.h
+++ b/components/password_manager/core/browser/password_form_manager.h
@@ -96,6 +96,11 @@ class PasswordFormManager : public PasswordStoreConsumer {
   bool HasGeneratedPassword();
   void SetHasGeneratedPassword();
 
+  void Update(

[vabr (Chromium), 2014/04/04 10:51:40]

This method definitely needs a comment explaining what it does, and what are the
arguments.

+    const base::string16& username_to_remove,
+    const PasswordStore::AuthorizationPromptPolicy& prompt_policy,
+    bool remove_candidate_from_store);
+
   // Determines if we need to autofill given the results of the query.
   // Takes ownership of the elements in |result|.
   void OnRequestDone(const std::vector<autofill::PasswordForm*>& result);
@@ -273,6 +278,9 @@ class PasswordFormManager : public PasswordStoreConsumer {
   // Whether this form has an auto generated password.
   bool has_generated_password_;
 
+  bool remove_candidate_from_store_;

[vabr (Chromium), 2014/04/04 10:51:40]

Given my suggestions in the *.cc file, these variables can be removed. (If we
end up not removing them, then we should at least comment on the purpose of
them.)

+  base::string16 username_to_remove_;
+
   // Set if the user has selected one of the other possible usernames in
   // |pending_credentials_|.
   base::string16 selected_username_;