Allow deleting autofill password suggestions on Shift+Delete

components/autofill/core/browser/password_autofill_manager.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 COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_
 #define COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_
 
 // This file was contains some repeated code from
 // chrome/renderer/autofill/password_autofill_manager because as we move to the
 // new Autofill UI we needs these functions in both the browser and renderer.
@@ skipping 18 matching lines @@
   // Fills the password associated with user name |username|. Returns true if
   // the username and password fields were filled, false otherwise.
   bool DidAcceptAutofillSuggestion(const FormFieldData& field,
                                    const base::string16& username);
 
   // Invoked when a password mapping is added.
   void AddPasswordFormMapping(
       const FormFieldData& username_element,
       const PasswordFormFillData& password);
 
+  // Clear the login associated with the field |field|

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

nit: this breaks too early (use the whole line)

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

"Clear the login" sounds ambiguous to me. What is it doing exactly? It looks
like staying close to the function name is less confusing: "Removes the password
suggestion associated..."

[rchtara, 2014/04/04 16:07:20]

Done.

[rchtara, 2014/04/04 16:07:20]

Done.

+  // and the user name |username_to_remove|. Returns true if the user name and
+  // password were removed.
+  bool RemovePasswordSuggestion(const FormFieldData& field,
+                                const base::string16& username_to_remove);
+
   // Invoked to clear any page specific cached values.
   void Reset();
 
  private:
   // TODO(csharp): Modify the AutofillExternalDeletegate code so that it can
   // figure out if a entry is a password one without using this mapping.
   // crbug.com/118601
   typedef std::map<FormFieldData,
                    PasswordFormFillData>
       LoginToPasswordInfoMap;
@@ skipping 13 matching lines @@
 
   // Provides driver-level context to the shared code of the component. Must
   // outlive |this|.
   AutofillDriver* const autofill_driver_;  // weak
 
   DISALLOW_COPY_AND_ASSIGN(PasswordAutofillManager);
 };
 
 }  // namespace autofill
 
-#endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_
+#endif // COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_