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

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

Issue 2748933005: Remove Obsolete HTTP SiteStatistics From PasswordStore during Migration (Closed)
Patch Set: Capture explicitly Created 3 years, 9 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/form_fetcher_impl.h
diff --git a/components/password_manager/core/browser/form_fetcher_impl.h b/components/password_manager/core/browser/form_fetcher_impl.h
index a9a1e7acae52d61a0dd6af7d9606d1ec63f38da6..af6ba69f03415c6d837f7e3d7aa602be69a82137 100644
--- a/components/password_manager/core/browser/form_fetcher_impl.h
+++ b/components/password_manager/core/browser/form_fetcher_impl.h
@@ -11,7 +11,7 @@
#include "base/macros.h"
#include "components/password_manager/core/browser/form_fetcher.h"
-#include "components/password_manager/core/browser/http_password_migrator.h"
+#include "components/password_manager/core/browser/http_password_store_migrator.h"
#include "components/password_manager/core/browser/password_store.h"
#include "components/password_manager/core/browser/password_store_consumer.h"
@@ -23,7 +23,7 @@ class PasswordManagerClient;
// with a particular origin.
class FormFetcherImpl : public FormFetcher,
public PasswordStoreConsumer,
- public HttpPasswordMigrator::Consumer {
+ public HttpPasswordStoreMigrator::Consumer {
public:
// |form_digest| describes what credentials need to be retrieved and
// |client| serves the PasswordStore, the logging information etc.
@@ -48,7 +48,7 @@ class FormFetcherImpl : public FormFetcher,
std::vector<std::unique_ptr<autofill::PasswordForm>> results) override;
void OnGetSiteStatistics(std::vector<InteractionsStats> stats) override;
- // HttpPasswordMigrator::Consumer:
+ // HttpPasswordStoreMigrator::Consumer:
void ProcessMigratedForms(
std::vector<std::unique_ptr<autofill::PasswordForm>> forms) override;
@@ -96,7 +96,7 @@ class FormFetcherImpl : public FormFetcher,
const bool should_migrate_http_passwords_;
// Does the actual migration.
- std::unique_ptr<HttpPasswordMigrator> http_migrator_;
+ std::unique_ptr<HttpPasswordStoreMigrator> http_migrator_;
DISALLOW_COPY_AND_ASSIGN(FormFetcherImpl);
};

Powered by Google App Engine
This is Rietveld 408576698