Allow deleting autofill password suggestions on Shift+Delete

components/autofill/core/browser/autofill_external_delegate.cc

Index: components/autofill/core/browser/autofill_external_delegate.cc
diff --git a/components/autofill/core/browser/autofill_external_delegate.cc b/components/autofill/core/browser/autofill_external_delegate.cc
index f435ceb8e23563face4edc083b1d815f12adcca7..d966a454dd0bdfc2cde37234e5c93392adc4e7ff 100644
--- a/components/autofill/core/browser/autofill_external_delegate.cc
+++ b/components/autofill/core/browser/autofill_external_delegate.cc
@@ -200,10 +200,15 @@ void AutofillExternalDelegate::DidAcceptSuggestion(const base::string16& value,
 
 void AutofillExternalDelegate::RemoveSuggestion(const base::string16& value,
                                                 int identifier) {
-  if (identifier > 0)
+  if (identifier > 0) {
     manager_->RemoveAutofillProfileOrCreditCard(identifier);
-  else
+  } else if (identifier == POPUP_ITEM_ID_PASSWORD_ENTRY) {
+    bool success =
+        password_manager_.RemovePasswordSuggestion(query_field_, value);
+    DCHECK(success);
+  } else {

[Ilya Sherman, 2014/04/09 06:33:05]

nit: Please DCHECK that the identifier is for an autocomplete entry.

[rchtara, 2014/04/09 13:52:43]

Done.

     manager_->RemoveAutocompleteEntry(query_field_.name, value);
+  }
 }
 
 void AutofillExternalDelegate::DidEndTextFieldEditing() {