Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1349)

Unified Diff: chrome/browser/password_manager/password_store.cc

Issue 133893004: Allow deleting autofill password suggestions on Shift+Delete (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: cleaning the code Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/password_manager/password_store.cc
diff --git a/chrome/browser/password_manager/password_store.cc b/chrome/browser/password_manager/password_store.cc
index d5880502c5ad7035e03995a6cd30e86578690c5c..7cdf58cb286ace356035dd4bccb6abe8fcd53c3b 100644
--- a/chrome/browser/password_manager/password_store.cc
+++ b/chrome/browser/password_manager/password_store.cc
@@ -39,6 +39,7 @@ void PostConsumerCallback(
const CancelableTaskTracker::IsCanceledCallback& is_canceled_cb,
PasswordStoreConsumer* consumer,
const base::Time& ignore_logins_cutoff,
+ const PasswordForm& form,
vabr (Chromium) 2014/03/07 23:43:36 Please indicate, what is the form needed for. Idea
riadh.chtara 2014/03/14 17:16:08 it's not needed. I forgot to remove it. Sorry for
const vector<PasswordForm*>& matched_forms) {
vector<PasswordForm*>* matched_forms_copy = new vector<PasswordForm*>();
if (ignore_logins_cutoff.is_null()) {
@@ -148,7 +149,8 @@ CancelableTaskTracker::TaskId PasswordStore::GetLogins(
base::MessageLoopProxy::current(),
is_canceled_cb,
consumer,
- ignore_logins_cutoff);
+ ignore_logins_cutoff,
+ form);
ScheduleTask(
base::Bind(&PasswordStore::GetLoginsImpl, this, form, callback_runner));
return id;

Powered by Google App Engine
This is Rietveld 408576698