Commit db38cd51 authored by Marc Treib's avatar Marc Treib Committed by Commit Bot

Cleanup: Remove unused PasswordStore::GetBlacklistLogins

Bug: none
Change-Id: I6d215536c172038cad55bf5a26f296149efe78f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100929
Commit-Queue: Marc Treib <treib@chromium.org>
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Auto-Submit: Marc Treib <treib@chromium.org>
Reviewed-by: default avatarMohamed Amir Yosef <mamir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#749777}
parent 8384e47e
...@@ -301,12 +301,6 @@ void PasswordStore::GetAutofillableLogins(PasswordStoreConsumer* consumer) { ...@@ -301,12 +301,6 @@ void PasswordStore::GetAutofillableLogins(PasswordStoreConsumer* consumer) {
base::BindOnce(&PasswordStore::GetAutofillableLoginsImpl, this)); base::BindOnce(&PasswordStore::GetAutofillableLoginsImpl, this));
} }
void PasswordStore::GetBlacklistLogins(PasswordStoreConsumer* consumer) {
DCHECK(main_task_runner_->RunsTasksInCurrentSequence());
PostLoginsTaskAndReplyToConsumerWithResult(
consumer, base::BindOnce(&PasswordStore::GetBlacklistLoginsImpl, this));
}
void PasswordStore::GetAllLogins(PasswordStoreConsumer* consumer) { void PasswordStore::GetAllLogins(PasswordStoreConsumer* consumer) {
DCHECK(main_task_runner_->RunsTasksInCurrentSequence()); DCHECK(main_task_runner_->RunsTasksInCurrentSequence());
PostLoginsTaskAndReplyToConsumerWithResult( PostLoginsTaskAndReplyToConsumerWithResult(
...@@ -1035,16 +1029,6 @@ PasswordStore::GetAutofillableLoginsImpl() { ...@@ -1035,16 +1029,6 @@ PasswordStore::GetAutofillableLoginsImpl() {
return obtained_forms; return obtained_forms;
} }
std::vector<std::unique_ptr<PasswordForm>>
PasswordStore::GetBlacklistLoginsImpl() {
DCHECK(background_task_runner_->RunsTasksInCurrentSequence());
TRACE_EVENT0("passwords", "PasswordStore::GetBlacklistLoginsImpl");
std::vector<std::unique_ptr<PasswordForm>> obtained_forms;
if (!FillBlacklistLogins(&obtained_forms))
obtained_forms.clear();
return obtained_forms;
}
std::vector<std::unique_ptr<PasswordForm>> PasswordStore::GetAllLoginsImpl() { std::vector<std::unique_ptr<PasswordForm>> PasswordStore::GetAllLoginsImpl() {
DCHECK(background_task_runner_->RunsTasksInCurrentSequence()); DCHECK(background_task_runner_->RunsTasksInCurrentSequence());
TRACE_EVENT0("passwords", "PasswordStore::GetAllLoginsImpl"); TRACE_EVENT0("passwords", "PasswordStore::GetAllLoginsImpl");
......
...@@ -227,11 +227,6 @@ class PasswordStore : protected PasswordStoreSync, ...@@ -227,11 +227,6 @@ class PasswordStore : protected PasswordStoreSync,
// The request will be cancelled if the consumer is destroyed. // The request will be cancelled if the consumer is destroyed.
virtual void GetAutofillableLogins(PasswordStoreConsumer* consumer); virtual void GetAutofillableLogins(PasswordStoreConsumer* consumer);
// Gets the complete list of PasswordForms that are blacklist entries and
// notifies |consumer| on completion. The request will be cancelled if the
// consumer is destroyed.
virtual void GetBlacklistLogins(PasswordStoreConsumer* consumer);
// Gets the complete list of PasswordForms (regardless of their blacklist // Gets the complete list of PasswordForms (regardless of their blacklist
// status) and notify |consumer| on completion. The request will be cancelled // status) and notify |consumer| on completion. The request will be cancelled
// if the consumer is destroyed. // if the consumer is destroyed.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment