Allow deleting autofill password suggestions on Shift+Delete

components/password_manager/core/browser/password_manager.cc

Index: components/password_manager/core/browser/password_manager.cc
diff --git a/components/password_manager/core/browser/password_manager.cc b/components/password_manager/core/browser/password_manager.cc
index 16ba3f4c78134008e21ea486775a7d0992c3d763..7ed5ba1cc0bc9d7f1b9a187c3380b3a707d42a4c 100644
--- a/components/password_manager/core/browser/password_manager.cc
+++ b/components/password_manager/core/browser/password_manager.cc
@@ -257,13 +257,56 @@ void PasswordManager::OnPasswordFormsParsed(
         this, client_, driver_, *iter, ssl_valid);
     pending_login_managers_.push_back(manager);
 
-    // Avoid prompting the user for access to a password if they don't have
-    // password saving enabled.
-    PasswordStore::AuthorizationPromptPolicy prompt_policy =
-        *password_manager_enabled_ ? PasswordStore::ALLOW_PROMPT
-                                   : PasswordStore::DISALLOW_PROMPT;
+    manager->FetchMatchingLoginsFromPasswordStore(GetPromptPolicy());
+  }
+}
+
+void PasswordManager::OnRemoveSavedPasswordAndUpdateManagers(
+    const PasswordForm& password_form) {
+  PasswordStore* password_store = client_->GetPasswordStore();
+  if (!password_store) {
+    NOTREACHED();
+    return;
+  }
+
+  // The |passwordform| doesn't have to much exactly the PasswordForm that is
+  // in the store. But both of them need to have the same username. Because
+  // an exact copy is needed to perform a deletion from the password store, and
+  // because every PasswordFormManager holds a copy of all the PasswordForms,
+  // this copy could be used in the deletion process.
 
-    manager->FetchMatchingLoginsFromPasswordStore(prompt_policy);
+  bool removed = false;
+  for (ScopedVector<PasswordFormManager>::iterator iter =
+           pending_login_managers_.begin();
+       iter != pending_login_managers_.end();
+       ++iter) {
+    // A PasswordFormManager that manages the |passwordform| AND where that
+    // has a PasswordForm with the key |password_form.username_value| in its
+    // best_matches_ needs to be find to get the PasswordForm we're looking
+    // for.
+    if ((*iter)->DoesManage(password_form,
+                            PasswordFormManager::ACTION_MATCH_REQUIRED)) {
+      PasswordFormMap best_matches = (*iter)->best_matches();
+      PasswordFormMap::iterator match =
+          best_matches.find(password_form.username_value);
+      if (match != best_matches.end()) {
+        password_store->RemoveLogin(*(match->second));
+        removed = true;
+        break;
+      }
+    }
+  }
+
+  if (!removed) {
+    NOTREACHED();
+    return;
+  }
+
+  for (ScopedVector<PasswordFormManager>::iterator iter =
+           pending_login_managers_.begin();
+       iter != pending_login_managers_.end();
+       ++iter) {
+    (*iter)->Update(GetPromptPolicy());
   }
 }
 
@@ -390,3 +433,8 @@ void PasswordManager::Autofill(
 
   client_->PasswordWasAutofilled(best_matches);
 }
+
+PasswordStore::AuthorizationPromptPolicy PasswordManager::GetPromptPolicy() {
+  return *password_manager_enabled_ ? PasswordStore::ALLOW_PROMPT
+                                    : PasswordStore::DISALLOW_PROMPT;
+}