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

Unified Diff: chrome/browser/component_updater/sw_reporter_installer_win.h

Issue 495643002: Trigger the SW reporter to run once a week for users with UMA enabled. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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: chrome/browser/component_updater/sw_reporter_installer_win.h
diff --git a/chrome/browser/component_updater/sw_reporter_installer_win.h b/chrome/browser/component_updater/sw_reporter_installer_win.h
index a3d0d4ebb56c8b56b1237cb5d2d1177c997d330f..6fda9447c0989be18f34f29171588bbd08b9c759 100644
--- a/chrome/browser/component_updater/sw_reporter_installer_win.h
+++ b/chrome/browser/component_updater/sw_reporter_installer_win.h
@@ -21,12 +21,13 @@ class ComponentUpdateService;
// be called from the UI thread.
void ExecuteSwReporter(ComponentUpdateService* cus, PrefService* prefs);
-// ExecutePending will only register/execute the SwReporter if a preference is
-// set in |prefs|, identifying that an attempt to register/execute the
-// SwReporter has already been made by calling ExecuteSwReporter above. But only
-// if the max number of retries has not been reached yet. This must be called
+// ExecuteSwReporterIfRequired will only register/execute the SwReporter if it
+// has been too long since it last ran or if a preference is set in |prefs|,
+// identifying that an attempt to register/execute the SwReporter has already
+// been made and not completed. This must be called
// from the UI thread.
-void ExecutePendingSwReporter(ComponentUpdateService* cus, PrefService* prefs);
+void ExecuteSwReporterIfRequired(ComponentUpdateService* cus,
+ PrefService* prefs);
// Register user preferences related to the SwReporter.
void RegisterPrefsForSwReporter(PrefRegistrySimple* registry);

Powered by Google App Engine
This is Rietveld 408576698