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

Unified Diff: components/password_manager/core/browser/password_store.h

Issue 2911433003: Refactor SuppressedHTTPSFormsFetcher to use GetLoginsForSameOrganizationName. (Closed)
Patch Set: Comment + rebase. Created 3 years, 7 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: components/password_manager/core/browser/password_store.h
diff --git a/components/password_manager/core/browser/password_store.h b/components/password_manager/core/browser/password_store.h
index 95983637243f35156cf1c192c84f99bdff0a180a..6c2f104e3264d528b78bb60a01ed341280e37e94 100644
--- a/components/password_manager/core/browser/password_store.h
+++ b/components/password_manager/core/browser/password_store.h
@@ -193,8 +193,9 @@ class PasswordStore : protected PasswordStoreSync,
// For example, the organization-identifying name of "https://foo.example.org"
// is `example`, and logins will be returned for "http://bar.example.co.uk",
// but not for "http://notexample.com" or "https://example.foo.com".
- void GetLoginsForSameOrganizationName(const std::string& signon_realm,
- PasswordStoreConsumer* consumer);
+ virtual void GetLoginsForSameOrganizationName(
+ const std::string& signon_realm,
+ PasswordStoreConsumer* consumer);
// Gets the complete list of PasswordForms that are not blacklist entries--and
// are thus auto-fillable. |consumer| will be notified on completion.

Powered by Google App Engine
This is Rietveld 408576698