Adding a SW reporter component updater

chrome/browser/profile_resetter/profile_resetter.cc

 // Copyright (c) 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 "chrome/browser/profile_resetter/profile_resetter.h"
 
 #include "base/prefs/pref_service.h"
 #include "base/prefs/scoped_user_pref_update.h"
 #include "base/synchronization/cancellation_flag.h"
+#include "chrome/browser/browser_process.h"
 #include "chrome/browser/browsing_data/browsing_data_helper.h"
 #include "chrome/browser/content_settings/host_content_settings_map.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/google/google_url_tracker_factory.h"
 #include "chrome/browser/profile_resetter/brandcoded_default_settings.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search_engines/template_url_prepopulate_data.h"
 #include "chrome/browser/search_engines/template_url_service.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_iterator.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/installer/util/browser_distribution.h"
 #include "components/google/core/browser/google_pref_names.h"
 #include "components/google/core/browser/google_url_tracker.h"
 #include "content/public/browser/browser_thread.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/browser/management_policy.h"
 
 #if defined(OS_WIN)
 #include "base/base_paths.h"
 #include "base/path_service.h"
+#include "chrome/browser/component_updater/sw_reporter_component_installer_win.h"
 #include "chrome/installer/util/shell_util.h"
 #include "content/public/browser/browser_thread.h"
 
 namespace {
 
 void ResetShortcutsOnFileThread() {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
   // Get full path of chrome.
   base::FilePath chrome_exe;
   if (!PathService::Get(base::FILE_EXE, &chrome_exe))
@@ skipping 27 matching lines @@
 }
 
 ProfileResetter::~ProfileResetter() {
   if (cookies_remover_)
     cookies_remover_->RemoveObserver(this);
 }
 
 void ProfileResetter::Reset(
     ProfileResetter::ResettableFlags resettable_flags,
     scoped_ptr<BrandcodedDefaultSettings> master_settings,
+    bool accepted_send_feedback,
     const base::Closure& callback) {
   DCHECK(CalledOnValidThread());
   DCHECK(master_settings);
 
   // We should never be called with unknown flags.
   CHECK_EQ(static_cast<ResettableFlags>(0), resettable_flags & ~ALL);
 
   // We should never be called when a previous reset has not finished.
   CHECK_EQ(static_cast<ResettableFlags>(0), pending_reset_flags_);
 
@@ skipping 24 matching lines @@
   };
 
   ResettableFlags reset_triggered_for_flags = 0;
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(flagToMethod); ++i) {
     if (resettable_flags & flagToMethod[i].flag) {
       reset_triggered_for_flags |= flagToMethod[i].flag;
       (this->*flagToMethod[i].method)();
     }
   }
 
+// When the user resets any of their settings on Windows, run the software

[battre, 2014/06/17 15:09:46]

... on Windows and agreed to sending feedback

[MAD, 2014/06/17 15:26:31]

Done.

+// reporter tool to see if it could find the reason why the user wanted a
+// reset.

[battre, 2014/06/17 15:09:46]

nit: +1 space

[MAD, 2014/06/17 15:26:31]

Where? At the beginning of the comments before the #if?

git cl format pushes it there...

+#if defined(OS_WIN)
+  // The browser process and / or local_state can be NULL when running tests.
+  if (accepted_send_feedback && g_browser_process &&
+      g_browser_process->local_state() &&
+      g_browser_process->local_state()->GetBoolean(
+          prefs::kMetricsReportingEnabled)) {
+    RegisterSwReporterComponent(g_browser_process->component_updater(),
+                                g_browser_process->local_state());
+  }
+#endif
+
   DCHECK_EQ(resettable_flags, reset_triggered_for_flags);
 }
 
 bool ProfileResetter::IsActive() const {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
   return pending_reset_flags_ != 0;
 }
 
 void ProfileResetter::MarkAsDone(Resettable resettable) {
   DCHECK(CalledOnValidThread());
@@ skipping 210 matching lines @@
         chrome_exe,
         false,
         cancel,
         &shortcuts);
   }
   return shortcuts;
 #else
   return std::vector<ShortcutCommand>();
 #endif
 }