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

Unified Diff: components/autofill/core/browser/personal_data_manager.cc

Issue 2061303002: [Autofill] Log the number of profiles considered/removed during dedupe. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: clean Created 4 years, 6 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/autofill/core/browser/personal_data_manager.cc
diff --git a/components/autofill/core/browser/personal_data_manager.cc b/components/autofill/core/browser/personal_data_manager.cc
index b62cbe6d6a81afcce7d33281e8491ffd87567e64..9e9bdc6b618e21a9d0768208c2b7ec228159cd32 100644
--- a/components/autofill/core/browser/personal_data_manager.cc
+++ b/components/autofill/core/browser/personal_data_manager.cc
@@ -1560,6 +1560,8 @@ void PersonalDataManager::FindAndMergeDuplicateProfiles(
const std::vector<AutofillProfile*>& existing_profiles,
AutofillProfile* profile_to_merge,
std::vector<std::string>* profile_guids_to_delete) {
+ AutofillMetrics::LogNumberOfProfilesConsideredForDedupe(
+ existing_profiles.size());
for (AutofillProfile* existing_profile : existing_profiles) {
// Don't try to merge a profile with itself or with any profile with a
// different PrimaryValue.
@@ -1580,6 +1582,9 @@ void PersonalDataManager::FindAndMergeDuplicateProfiles(
}
}
}
+
+ AutofillMetrics::LogNumberOfProfilesRemovedDuringDedupe(
+ profile_guids_to_delete->size());
}
} // namespace autofill

Powered by Google App Engine
This is Rietveld 408576698