Primary histograms for InBrowserCleanerUI experiment

chrome/browser/safe_browsing/chrome_cleaner/settings_resetter_win.cc

 // Copyright 2017 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 "chrome/browser/safe_browsing/chrome_cleaner/settings_resetter_win.h"
 
 #include <algorithm>
 #include <iterator>
 #include <memory>
 #include <utility>
 #include <vector>
 
 #include "base/barrier_closure.h"
 #include "base/bind_helpers.h"
 #include "base/callback_helpers.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
+#include "base/metrics/histogram_macros.h"
 #include "base/sequence_checker.h"
 #include "base/synchronization/lock.h"
 #include "base/win/registry.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profile_resetter/profile_resetter.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/safe_browsing/chrome_cleaner/srt_field_trial_win.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/common/pref_names.h"
 #include "components/chrome_cleaner/public/constants/constants.h"
 #include "components/pref_registry/pref_registry_syncable.h"
 #include "components/prefs/pref_service.h"
 #include "content/public/browser/browser_thread.h"
 
 namespace safe_browsing {
 
 namespace {
 
+enum TaggedProfileForResetting {

[csharp, 2017/07/07 20:53:33]

nit: Add a comment about matching uma

[ftirelo, 2017/07/07 21:22:08]

Done.

+  TAGGED_PROFILE_FOR_RESETTING_TAGGED = 0,
+
+  TAGGED_PROFILE_FOR_RESETTING_MAX,
+};
+
 // Returns the post-cleanup reset pending prefs for |profile|.
 bool ResetPending(Profile* profile) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   DCHECK(PostCleanupSettingsResetter::IsEnabled());
   DCHECK(profile);
 
   PrefService* prefs = profile->GetPrefs();
   return prefs->GetBoolean(prefs::kChromeCleanerResetPending);
 }
 
@@ skipping 165 matching lines @@
 PostCleanupSettingsResetter::Delegate::GetProfileResetter(Profile* profile) {
   return base::MakeUnique<ProfileResetter>(profile);
 }
 
 void PostCleanupSettingsResetter::TagForResetting(Profile* profile) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   DCHECK(IsEnabled());
   DCHECK(profile);
 
   RecordResetPending(true, profile);
+  UMA_HISTOGRAM_ENUMERATION("SoftwareReporter.TaggedProfileForResetting",
+                            TAGGED_PROFILE_FOR_RESETTING_TAGGED,
+                            TAGGED_PROFILE_FOR_RESETTING_MAX);
 }
 
 void PostCleanupSettingsResetter::ResetTaggedProfiles(
     std::vector<Profile*> profiles,
     base::OnceClosure done_callback,
     std::unique_ptr<PostCleanupSettingsResetter::Delegate> delegate) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   DCHECK(IsEnabled());
   DCHECK(delegate);
 
   std::vector<Profile*> profiles_to_reset;
   if (!CopyProfilesToReset(profiles, &profiles_to_reset) ||
       !CleanupCompletedFromRegistry()) {
     std::move(done_callback).Run();
     return;
   }
 
+  UMA_HISTOGRAM_EXACT_LINEAR("SoftwareReporter.PostCleanupSettingsReset",
+                             profiles_to_reset.size(), 10);
+
   // The SettingsResetter object will self-delete once |done_callback| is
   // invoked.
   make_scoped_refptr(new SettingsResetter(std::move(profiles_to_reset),
                                           std::move(delegate),
                                           std::move(done_callback)))
       ->Run();
 }
 
 // static
 void PostCleanupSettingsResetter::RegisterProfilePrefs(
     user_prefs::PrefRegistrySyncable* registry) {
   DCHECK(registry);
   registry->RegisterBooleanPref(prefs::kChromeCleanerResetPending, false);
 }
 
 }  // namespace safe_browsing