Trigger the SW reporter to run once a week for users with UMA enabled.

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 <string>
 
 #include "base/prefs/pref_service.h"
 #include "base/prefs/scoped_user_pref_update.h"
@@ skipping 16 matching lines @@
 #include "components/search_engines/search_engines_pref_names.h"
 #include "components/search_engines/template_url_prepopulate_data.h"
 #include "components/search_engines/template_url_service.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_installer_win.h"

[MAD, 2014/08/22 18:14:51]

You should also be able to remove 
#include "chrome/browser/browser_process.h"

[csharp, 2014/08/22 19:39:22]

Done.

 #include "chrome/installer/util/shell_util.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))
     return;
@@ skipping 71 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 and agreed to sending
-// feedback, run the software reporter tool to see if it could find the reason
-// why the user wanted a reset.
-#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)) {
-    ExecuteSwReporter(g_browser_process->component_updater(),
-                      g_browser_process->local_state());
-  }
-#endif
-

[waffles, 2014/08/22 18:52:53]

With this code now gone, how does a user initially acquire SwReporter? If I'm
not mistaken, ExecuteSwReporterIfRequired will only register the component if
there is a SwReporter already on disk, and this was the only call to
ExecuteSwReporter that didn't go through ExecuteSwReporterIfRequired.

[MAD, 2014/08/22 19:02:09]

ExecuteSwReporter is now called when it wasn't called in the last 7 days, as
added in ExecuteSwReporterIfRequired() which replaced ExecutePendingSwReporter
in sw_reporter_installer_win.cc

   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 211 matching lines @@
         chrome_exe,
         false,
         cancel,
         &shortcuts);
   }
   return shortcuts;
 #else
   return std::vector<ShortcutCommand>();
 #endif
 }