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

Unified Diff: components/password_manager/core/browser/test_password_store.cc

Issue 1200603003: Integrate PasswordStoreProxyMac instead of PasswordStoreMac. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments Created 5 years, 6 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
« no previous file with comments | « components/password_manager/core/browser/test_password_store.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/password_manager/core/browser/test_password_store.cc
diff --git a/components/password_manager/core/browser/test_password_store.cc b/components/password_manager/core/browser/test_password_store.cc
index b54b99c82edb11c6581f431230ffb0781f557f0f..b1994d1899d0079570fc64c037d468a21cd0444d 100644
--- a/components/password_manager/core/browser/test_password_store.cc
+++ b/components/password_manager/core/browser/test_password_store.cc
@@ -46,16 +46,6 @@ bool TestPasswordStore::FormsAreEquivalent(const autofill::PasswordForm& lhs,
lhs.signon_realm == rhs.signon_realm;
}
-void TestPasswordStore::GetAutofillableLoginsImpl(
- scoped_ptr<GetLoginsRequest> request) {
- ScopedVector<autofill::PasswordForm> results;
- for (const auto& forms_for_realm : stored_passwords_) {
- for (const autofill::PasswordForm& form : forms_for_realm.second)
- results.push_back(new autofill::PasswordForm(form));
- }
- request->NotifyConsumerWithResults(results.Pass());
-}
-
PasswordStoreChangeList TestPasswordStore::AddLoginImpl(
const autofill::PasswordForm& form) {
PasswordStoreChangeList changes;
@@ -126,12 +116,12 @@ PasswordStoreChangeList TestPasswordStore::RemoveLoginsSyncedBetweenImpl(
return changes;
}
-void TestPasswordStore::GetBlacklistLoginsImpl(
- scoped_ptr<GetLoginsRequest> request) {
-}
-
bool TestPasswordStore::FillAutofillableLogins(
ScopedVector<autofill::PasswordForm>* forms) {
+ for (const auto& forms_for_realm : stored_passwords_) {
+ for (const autofill::PasswordForm& form : forms_for_realm.second)
+ forms->push_back(new autofill::PasswordForm(form));
+ }
return true;
}
« no previous file with comments | « components/password_manager/core/browser/test_password_store.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698