Move Credentials when migrating to HSTS page

components/password_manager/core/browser/http_password_migrator.cc

Index: components/password_manager/core/browser/http_password_migrator.cc
diff --git a/components/password_manager/core/browser/http_password_migrator.cc b/components/password_manager/core/browser/http_password_migrator.cc
index 700327ca480668075b82f08b08d7c769c9d630d9..7be6801acfbc06c93edc4395f854b1542e4d15d0 100644
--- a/components/password_manager/core/browser/http_password_migrator.cc
+++ b/components/password_manager/core/browser/http_password_migrator.cc
@@ -65,6 +65,10 @@ void HttpPasswordMigrator::OnGetPasswordStoreResults(
   }
 
   metrics_util::LogCountHttpMigratedPasswords(results.size());
+  metrics_util::LogHttpPasswordMigrationMode(
+      mode_ == MigrationMode::MOVE
+          ? metrics_util::HTTP_PASSWORD_MIGRATION_MODE_MOVE

[jdoerrie, 2017/02/27 18:33:12]

Having two MigrationMode enums feels kinda redundant, but I didn't feel
comfortable logging our enum directly to UMA. Also having two different enums
enables having a _COUNT member, so that in case more modes would be added at
some point we have to change less code. Not sure how relevant the last point is,
though. WDYT?

[vasilii, 2017/02/28 09:10:37]

You did everything right.

+          : metrics_util::HTTP_PASSWORD_MIGRATION_MODE_COPY);
 
   if (consumer_)
     consumer_->ProcessMigratedForms(std::move(results));