Minor style cleanup in metrics_reporting_scheduler.*

chrome/browser/metrics/metrics_reporting_scheduler.cc

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 67caa73bf2bcddc472647c65e98fdd7e816c5f0e..d56db79ace7071d870f50e2a6652c4383375cebf 100644
--- a/chrome/browser/metrics/metrics_reporting_scheduler.cc
+++ b/chrome/browser/metrics/metrics_reporting_scheduler.cc
@@ -7,27 +7,26 @@
 #include "base/compiler_specific.h"
 #include "chrome/browser/metrics/metrics_service.h"
 
-using base::Time;
 using base::TimeDelta;
 
 // The delay, in seconds, after startup before sending the first log message.
-static const int kInitialUploadIntervalSeconds = 60;

[jar (doing other things), 2012/06/08 00:20:02]

I'm surprised you pulled the static qualifier. Won't that cause the vars to have
extern linkage (i.e., be visible at the linker level)?

Usually, we either use an anonymous namespace, or static, to preclude that.

[Ilya Sherman, 2012/06/08 00:52:14]

Numeric constants don't seem to require either, but I've gone ahead and wrapped
these in an anonymous namespace to be extra sure.  (I know, I could probably
just leave the "static"... but that doesn't match the style I've seen in other
files, and so I find it surprising when reading through the code.)

[jar (doing other things), 2012/06/11 18:54:28]

If I had my druthers.... I'd suggest that Chromium use the static qualifier, as
it makes it easier to grep the code etc. ...but no one asked me ;-).

The only restriction is that static doesn't work for classes <sigh>, so for them
you need the anonymous namespace.

On 2012/06/08 00:52:14, Ilya Sherman wrote:

+const int kInitialUploadIntervalSeconds = 60;
 
 // The delay, in seconds, between uploading when there are queued logs from
 // previous sessions to send.
-static const int kUnsentLogsIntervalSeconds = 15;
+const int kUnsentLogsIntervalSeconds = 15;
 
 // Standard interval between log uploads, in seconds.
-static const int kStandardUploadIntervalSeconds = 30 * 60;  // Thirty minutes.
+const int kStandardUploadIntervalSeconds = 30 * 60;  // Thirty minutes.
 
 // When uploading metrics to the server fails, we progressively wait longer and
 // longer before sending the next log. This backoff process helps reduce load
 // on a server that is having issues.
 // The following is the multiplier we use to expand that inter-log duration.
-static const double kBackoffMultiplier = 1.1;
+const double kBackoffMultiplier = 1.1;
 
 // The maximum backoff multiplier.
-static const int kMaxBackoffMultiplier = 10;
+const int kMaxBackoffMultiplier = 10;
 
 
 MetricsReportingScheduler::MetricsReportingScheduler(
@@ -42,7 +41,7 @@ MetricsReportingScheduler::~MetricsReportingScheduler() {}
 
 void MetricsReportingScheduler::Start() {
   running_ = true;
-  ScheduleNextCallback();
+  ScheduleNextUpload();
 }
 
 void MetricsReportingScheduler::Stop() {
@@ -67,14 +66,14 @@ void MetricsReportingScheduler::UploadFinished(bool server_is_healthy,
   }
 
   if (running_)
-    ScheduleNextCallback();
+    ScheduleNextUpload();
 }
 
 void MetricsReportingScheduler::UploadCancelled() {
   DCHECK(callback_pending_);
   callback_pending_ = false;
   if (running_)
-    ScheduleNextCallback();
+    ScheduleNextUpload();
 }
 
 void MetricsReportingScheduler::TriggerUpload() {
@@ -82,7 +81,7 @@ void MetricsReportingScheduler::TriggerUpload() {
   upload_callback_.Run();
 }
 
-void MetricsReportingScheduler::ScheduleNextCallback() {
+void MetricsReportingScheduler::ScheduleNextUpload() {
   DCHECK(running_);
   if (upload_timer_.IsRunning() || callback_pending_)
     return;
@@ -92,7 +91,7 @@ void MetricsReportingScheduler::ScheduleNextCallback() {
 }
 
 void MetricsReportingScheduler::BackOffUploadInterval() {
-  DCHECK(kBackoffMultiplier > 1.0);
+  DCHECK_GT(kBackoffMultiplier, 1.0);
   upload_interval_ = TimeDelta::FromMicroseconds(
       static_cast<int64>(kBackoffMultiplier *
                          upload_interval_.InMicroseconds()));