Histogram argument related change

chrome/browser/chromeos/external_metrics.cc

Index: chrome/browser/chromeos/external_metrics.cc
diff --git a/chrome/browser/chromeos/external_metrics.cc b/chrome/browser/chromeos/external_metrics.cc
index 927761657e104b71458c209e9b2e2a4c682fb114..0f0d6b4360ba3386555f1499a029fd055cd81ed3 100644
--- a/chrome/browser/chromeos/external_metrics.cc
+++ b/chrome/browser/chromeos/external_metrics.cc
@@ -35,7 +35,7 @@ namespace chromeos {
 // The interval between external metrics collections in seconds
 static const int kExternalMetricsCollectionIntervalSeconds = 30;
 
-class SystemHistogram : public base::Histogram {

[zel, 2012/10/24 01:11:00]

You can get rid of this class now and just expose CheckValues() and
CheckLinearValues() as functions.

[kaiwang, 2012/10/24 03:43:28]

Done.

+class SystemHistogram {
  public:
   static bool CheckValues(const std::string& name,
                           int minimum,
@@ -44,7 +44,7 @@ class SystemHistogram : public base::Histogram {
     if (!base::Histogram::InspectConstructionArguments(
         name, &minimum, &maximum, &bucket_count))
       return false;
-    Histogram* histogram = base::StatisticsRecorder::FindHistogram(name);
+    HistogramBase* histogram = base::StatisticsRecorder::FindHistogram(name);
     if (!histogram)
       return true;
     return histogram->HasConstructionArguments(minimum, maximum, bucket_count);