Fix sync-credential-related metrics

components/password_manager/core/browser/password_form_manager_unittest.cc

Index: components/password_manager/core/browser/password_form_manager_unittest.cc
diff --git a/components/password_manager/core/browser/password_form_manager_unittest.cc b/components/password_manager/core/browser/password_form_manager_unittest.cc
index b294ad38ca682431fa2fede85b230a75e3c20295..2b9bf5be2f8beae7ba96436dfcb800ab47e17e57 100644
--- a/components/password_manager/core/browser/password_form_manager_unittest.cc
+++ b/components/password_manager/core/browser/password_form_manager_unittest.cc
@@ -14,6 +14,7 @@
 #include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/test/histogram_tester.h"
+#include "base/test/user_action_tester.h"
 #include "build/build_config.h"
 #include "components/autofill/core/browser/autofill_manager.h"
 #include "components/autofill/core/browser/proto/server.pb.h"
@@ -3026,4 +3027,19 @@ TEST_F(PasswordFormManagerFillOnAccountSelectTest, ProcessFrame) {
   SimulateMatchingPhase(form_manager(), RESULT_SAVED_MATCH);
 }
 
+// Check that PasswordFormManager records PasswordManager_ProcessingUpdate as
+// part of processing a credential update.
+TEST_F(PasswordFormManagerTest, ReportProcessingUpdate) {
+  SimulateMatchingPhase(form_manager(), RESULT_SAVED_MATCH);
+  form_manager()->ProvisionallySave(
+      *saved_match(), PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
+
+  EXPECT_FALSE(form_manager()->IsNewLogin());
+
+  base::UserActionTester tester;
+  EXPECT_EQ(0, tester.GetActionCount("PasswordManager_ProcessingUpdate"));
+  form_manager()->Update(*saved_match());
+  EXPECT_EQ(1, tester.GetActionCount("PasswordManager_ProcessingUpdate"));
+}
+
 }  // namespace password_manager