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

Unified Diff: chrome/browser/metrics/metrics_reporting_scheduler.cc

Issue 7001003: Switch MetricsReportingScheduler to using an actual timer object (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Updating to trunk Created 8 years, 10 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
« no previous file with comments | « chrome/browser/metrics/metrics_reporting_scheduler.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/metrics/metrics_reporting_scheduler.cc
diff --git a/chrome/browser/metrics/metrics_reporting_scheduler.cc b/chrome/browser/metrics/metrics_reporting_scheduler.cc
index 05ff108381b459114fde6848899aa5a04c5e1f64..67caa73bf2bcddc472647c65e98fdd7e816c5f0e 100644
--- a/chrome/browser/metrics/metrics_reporting_scheduler.cc
+++ b/chrome/browser/metrics/metrics_reporting_scheduler.cc
@@ -33,10 +33,8 @@ static const int kMaxBackoffMultiplier = 10;
MetricsReportingScheduler::MetricsReportingScheduler(
const base::Closure& upload_callback)
: upload_callback_(upload_callback),
- ALLOW_THIS_IN_INITIALIZER_LIST(weak_ptr_factory_(this)),
upload_interval_(TimeDelta::FromSeconds(kInitialUploadIntervalSeconds)),
running_(false),
- timer_pending_(false),
callback_pending_(false) {
}
@@ -49,6 +47,8 @@ void MetricsReportingScheduler::Start() {
void MetricsReportingScheduler::Stop() {
running_ = false;
+ if (upload_timer_.IsRunning())
+ upload_timer_.Stop();
}
void MetricsReportingScheduler::UploadFinished(bool server_is_healthy,
@@ -78,23 +78,17 @@ void MetricsReportingScheduler::UploadCancelled() {
}
void MetricsReportingScheduler::TriggerUpload() {
- timer_pending_ = false;
callback_pending_ = true;
upload_callback_.Run();
}
void MetricsReportingScheduler::ScheduleNextCallback() {
DCHECK(running_);
- if (timer_pending_ || callback_pending_)
+ if (upload_timer_.IsRunning() || callback_pending_)
return;
- timer_pending_ = true;
-
- MessageLoop::current()->PostDelayedTask(
- FROM_HERE,
- base::Bind(&MetricsReportingScheduler::TriggerUpload,
- weak_ptr_factory_.GetWeakPtr()),
- upload_interval_);
+ upload_timer_.Start(FROM_HERE, upload_interval_, this,
+ &MetricsReportingScheduler::TriggerUpload);
}
void MetricsReportingScheduler::BackOffUploadInterval() {
« no previous file with comments | « chrome/browser/metrics/metrics_reporting_scheduler.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698