Refactor histogram_macros.h. This improves documentation for the macros in the file, moves LOCAL_* …

base/metrics/histogram_base.h

Index: base/metrics/histogram_base.h
diff --git a/base/metrics/histogram_base.h b/base/metrics/histogram_base.h
index d240099110e8bf66cf64d8a140cd860471880d5f..c699b68ba1bb7c2bbe518f3a80b759d53054dad3 100644
--- a/base/metrics/histogram_base.h
+++ b/base/metrics/histogram_base.h
@@ -92,7 +92,7 @@ class BASE_EXPORT HistogramBase {
   static const Sample kSampleType_MAX;  // INT_MAX
 
   enum Flags {
-    kNoFlags = 0,
+    kNoFlags = 0x0,
 
     // Histogram should be UMA uploaded.
     kUmaTargetedHistogramFlag = 0x1,
@@ -122,6 +122,8 @@ class BASE_EXPORT HistogramBase {
     // histogram is created.
     kIsPersistent = 0x40,
 
+    // TODO(rkaplow): Look into this, but looks like this is unused and can
+    // be removed.

[Alexei Svitkine (slow), 2016/09/26 20:35:47]

FYI: I disagree with the earlier comment about this about resolving this TODO in
this CL. I think this CL is already doing a lot so I support adding the TODO
here and looking at this in a follow up CL.

     // Only for Histogram and its sub classes: fancy bucket-naming support.
     kHexRangePrintingFlag = 0x8000,
   };