Histogram type support in HistogramBase and remove SetRangeDescription function

base/metrics/histogram_base.cc

Index: base/metrics/histogram_base.cc
diff --git a/base/metrics/histogram_base.cc b/base/metrics/histogram_base.cc
index a5fade959eb41859d23133a132dc8f856bb6bf31..487d796a5a2b0adc2f9cc8a0243a389971f1e1f1 100644
--- a/base/metrics/histogram_base.cc
+++ b/base/metrics/histogram_base.cc
@@ -6,12 +6,37 @@
 
 #include <climits>
 
+#include "base/logging.h"
 #include "base/json/json_string_value_serializer.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 
 namespace base {
 
+std::string HistogramTypeName(HistogramType type) {
+  switch(type) {
+    case HISTOGRAM:
+      return "HISTOGRAM";
+      break;
+    case LINEAR_HISTOGRAM:
+      return "LINEAR_HISTOGRAM";
+      break;
+    case BOOLEAN_HISTOGRAM:
+      return "BOOLEAN_HISTOGRAM";
+      break;
+    case CUSTOM_HISTOGRAM:
+      return "CUSTOM_HISTOGRAM";
+      break;
+    case SPARSE_HISTOGRAM:
+      return "SPARSE_HISTOGRAM";
+      break;
+    default:
+      NOTREACHED();
+      break;

[Ilya Sherman, 2012/10/31 00:35:19]

nit: No need for any break stmts in this switch, since each case has a return
stmt, except for the default case which is the last case.

[kaiwang, 2012/10/31 01:12:56]

Done.

+  }
+  return "UNKNOWN";
+}
+
 const HistogramBase::Sample HistogramBase::kSampleType_MAX = INT_MAX;
 
 HistogramBase::HistogramBase(const std::string& name)