[Autofill] Log the number of profiles considered/removed during dedupe.

components/autofill/core/browser/personal_data_manager_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/autofill/core/browser/personal_data_manager.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <list>
 #include <map>
 #include <memory>
 #include <string>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/guid.h"
 #include "base/metrics/field_trial.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/synchronization/waitable_event.h"
+#include "base/test/histogram_tester.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/time/time.h"
 #include "build/build_config.h"
 #include "components/autofill/core/browser/autofill_experiments.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/autofill_test_utils.h"
 #include "components/autofill/core/browser/field_types.h"
 #include "components/autofill/core/browser/form_structure.h"
 #include "components/autofill/core/browser/personal_data_manager_observer.h"
 #include "components/autofill/core/browser/webdata/autofill_table.h"
@@ skipping 4401 matching lines @@
   EXPECT_EQ(4U, personal_data_->GetProfiles().size());
 
   // Create a new imported profile with no company name. It is similar to
   // profiles 1 and 2 and should be merged with them.
   AutofillProfile imported_profile(base::GenerateGUID(),
                                    "https://www.example.com");
   test::SetProfileInfo(&imported_profile, "Homer", "Jay", "Simpson",
                        "homer.simpson@abc.com", "", "742. Evergreen Terrace",
                        "", "Springfield", "IL", "91601", "US", "12345678910");
 
+  base::HistogramTester histogram_tester;
   // Save the imported profile (use it).
   personal_data_->SaveImportedProfile(imported_profile);
 
   EXPECT_CALL(personal_data_observer_, OnPersonalDataChanged())
       .WillOnce(QuitMainMessageLoop());
   base::MessageLoop::current()->Run();
 
   std::vector<AutofillProfile*> profiles = personal_data_->GetProfiles();
 
   // The imported profile and saved profiles 1 and 2 should be merged together.
   // Therefore there should only be 3 saved profiles.
   ASSERT_EQ(3U, profiles.size());
+  // 4 profiles were considered for dedupe.
+  histogram_tester.ExpectUniqueSample(
+      "Autofill.NumberOfProfilesConsideredForDedupe", 4, 1);
+  // 1 profile was removed.
+  histogram_tester.ExpectUniqueSample(
+      "Autofill.NumberOfProfilesRemovedDuringDedupe", 1, 1);
 
   // Sort the profiles by frecency to have a deterministic order.
   base::Time comparison_time = base::Time::Now();
   std::sort(profiles.begin(), profiles.end(),
             [comparison_time](const AutofillDataModel* a,
                               const AutofillDataModel* b) {
               return a->CompareFrecency(b, comparison_time);
             });
 
   // Since profiles with higher frecency scores are merged into profiles with
@@ skipping 57 matching lines @@
                        "homer.simpson@abc.com", "Fox", "742 Evergreen Terrace.",
                        "", "Springfield", "IL", "91601", "US", "12345678910");
 
   std::vector<AutofillProfile*> existing_profiles;
   existing_profiles.push_back(&profile1);
   existing_profiles.push_back(&profile2);
   existing_profiles.push_back(&profile3);
   existing_profiles.push_back(&profile4);
   existing_profiles.push_back(&profile5);
 
+  base::HistogramTester histogram_tester;
   std::vector<std::string> guids_to_delete;
   personal_data_->FindAndMergeDuplicateProfiles(existing_profiles, &profile1,
                                                 &guids_to_delete);
+  // 5 profiles were considered for dedupe.
+  histogram_tester.ExpectUniqueSample(
+      "Autofill.NumberOfProfilesConsideredForDedupe", 5, 1);
+  // 2 profiles were removed.
+  histogram_tester.ExpectUniqueSample(
+      "Autofill.NumberOfProfilesRemovedDuringDedupe", 2, 1);
 
   // Profile1 should be deleted because it was sent as the profile to merge and
   // thus was merged into profile3 and then into profile5.
   EXPECT_TRUE(std::find(guids_to_delete.begin(), guids_to_delete.end(),
                         profile1.guid()) != guids_to_delete.end());
 
   // Profile3 should be deleted because profile1 was merged into it and the
   // resulting profile was then merged into profile5.
   EXPECT_TRUE(std::find(guids_to_delete.begin(), guids_to_delete.end(),
                         profile3.guid()) != guids_to_delete.end());
@@ skipping 35 matching lines @@
 
   EXPECT_EQ(2U, personal_data_->GetProfiles().size());
 
   // Create a new imported profile to be merged with the saved profiles.
   AutofillProfile imported_profile(base::GenerateGUID(),
                                    "https://www.example.com");
   test::SetProfileInfo(&imported_profile, "Homer", "J", "Simpson",
                        "homer.simpson@abc.com", "", "742. Evergreen Terrace",
                        "", "Springfield", "IL", "91601", "US", "");
 
+  base::HistogramTester histogram_tester;
   personal_data_->SaveImportedProfile(imported_profile);
 
   EXPECT_CALL(personal_data_observer_, OnPersonalDataChanged())
       .WillOnce(QuitMainMessageLoop());
   base::MessageLoop::current()->Run();
 
   std::vector<AutofillProfile*> profiles = personal_data_->GetProfiles();
 
   // The imported profile and saved profiles 1 and 2 should be merged together.
   // Therefore there should only be 1 saved profile.
   ASSERT_EQ(1U, profiles.size());
+  // 2 profiles were considered for dedupe.
+  histogram_tester.ExpectUniqueSample(
+      "Autofill.NumberOfProfilesConsideredForDedupe", 2, 1);
+  // 1 profile was removed.
+  histogram_tester.ExpectUniqueSample(
+      "Autofill.NumberOfProfilesRemovedDuringDedupe", 1, 1);
 
   // Since profiles with higher frecency scores are merged into profiles with
   // lower frecency scores, the result of the merge should be contained in
   // profile2 since it had a lower frecency score compared to profile1.
   EXPECT_EQ(profile2.guid(), profiles[0]->guid());
   // The address syntax that results from the merge should be the one from the
   // imported profile (highest frecency).
   EXPECT_EQ(UTF8ToUTF16("742. Evergreen Terrace"),
             profiles[0]->GetRawInfo(ADDRESS_HOME_LINE1));
   // The middle name should be full, even if the profile with the higher
@@ skipping 14 matching lines @@
   EXPECT_EQ(profile1.use_count() + profile2.use_count() +
                 imported_profile.use_count(),
             profiles[0]->use_count());
   // The use date that results from the merge should be the one from the
   // imported profile since it was used just now.
   EXPECT_LT(base::Time::Now() - base::TimeDelta::FromSeconds(10),
             profiles[0]->use_date());
 }
 
 }  // namespace autofill