Additional logging for hash saving.

Bug: 994291
Change-Id: I48fe790dbf6c686e83264b638b0b031eb801d626
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762239Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Vadym Doroshenko <dvadym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#688605}

components/autofill/core/common/save_password_progress_logger.cc

@@ -517,6 +517,12 @@ std::string SavePasswordProgressLogger::GetStringFromID(
     case SavePasswordProgressLogger::
         STRING_PASSWORD_REQUIREMENTS_VOTE_NO_PASSWORD_ATTRIBUTES:
       return "No password requirements attributed set";
+    case STRING_SAVE_PASSWORD_HASH:
+      return "Password hash is saved";
+    case STRING_DID_NAVIGATE_MAIN_FRAME:
+      return "PasswordManager::DidNavigateMainFrame";
+    case STRING_NAVIGATION_NTP:
+      return "Navigation to New Tab page";
     case SavePasswordProgressLogger::STRING_INVALID:
       return "INVALID";
       // Intentionally no default: clause here -- all IDs need to get covered.

components/autofill/core/common/save_password_progress_logger.h

@@ -191,6 +191,9 @@ class SavePasswordProgressLogger {
     STRING_PASSWORD_REQUIREMENTS_VOTE_FOR_SPECIFIC_SPECIAL_SYMBOL,
     STRING_PASSWORD_REQUIREMENTS_VOTE_FOR_PASSWORD_LENGTH,
     STRING_PASSWORD_REQUIREMENTS_VOTE_NO_PASSWORD_ATTRIBUTES,
+    STRING_SAVE_PASSWORD_HASH,
+    STRING_DID_NAVIGATE_MAIN_FRAME,
+    STRING_NAVIGATION_NTP,
     STRING_INVALID,  // Represents a string returned in a case of an error.
     STRING_MAX = STRING_INVALID
   };

components/password_manager/core/browser/password_manager.cc

@@ -569,9 +569,19 @@ void PasswordManager::ProvisionallySavePassword(
 }
 
 void PasswordManager::DidNavigateMainFrame(bool form_may_be_submitted) {
+  std::unique_ptr<BrowserSavePasswordProgressLogger> logger;
+  if (password_manager_util::IsLoggingActive(client_)) {
+    logger.reset(
+        new BrowserSavePasswordProgressLogger(client_->GetLogManager()));
+    logger->LogBoolean(Logger::STRING_DID_NAVIGATE_MAIN_FRAME,
+                       form_may_be_submitted);
+  }
+
   pending_login_managers_.clear();
 
   if (client_->IsNewTabPage()) {
+    if (logger)
+      logger->LogMessage(Logger::STRING_NAVIGATION_NTP);
     // On a successful Chrome sign-in the page navigates to the new tab page
     // (ntp). OnPasswordFormsRendered is not called on ntp. That is why the
     // standard flow for saving hash does not work. Save a password hash now
@@ -1293,6 +1303,11 @@ void PasswordManager::MaybeSavePasswordHash(
   if (!should_save_enterprise_pw && !should_save_gaia_pw)
     return;
 
+  if (password_manager_util::IsLoggingActive(client_)) {
+    BrowserSavePasswordProgressLogger logger(client_->GetLogManager());
+    logger.LogMessage(Logger::STRING_SAVE_PASSWORD_HASH);
+  }
+
   // Canonicalizes username if it is an email.
   if (username.find('@') != std::string::npos)
     username = gaia::CanonicalizeEmail(username);