Increase Max Range for RasterPixelsPerMs

cc/base/histograms.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CC_BASE_HISTOGRAMS_H_
 #define CC_BASE_HISTOGRAMS_H_
 
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/metrics/histogram_base.h"
@@ skipping 46 matching lines @@
   class_name::~class_name() {                                               \
     Sample time_sample;                                                     \
     Sample area_sample;                                                     \
     const char* client_name = GetClientNameForMetrics();                    \
     if (client_name && GetHistogramValues(&time_sample, &area_sample)) {    \
       /* GetClientNameForMetrics only returns one non-null value over */    \
       /* the lifetime of the process, so these histogram names are */       \
       /* runtime constant. */                                               \
       UMA_HISTOGRAM_COUNTS(base::StringPrintf(time_histogram, client_name), \
                            time_sample);                                    \
-      UMA_HISTOGRAM_COUNTS(base::StringPrintf(area_histogram, client_name), \
-                           area_sample);                                    \
+      UMA_HISTOGRAM_CUSTOM_COUNTS(                                          \
+          base::StringPrintf(time_histogram, client_name), area_sample, 1,  \
+          100000000, 50);                                                   \

[ericrk, 2016/11/14 17:33:32]

isherman@ - when we update the range of an existing histogram, do we typically
rename (add 2, etc...) to prevent confusion? Or just add a note for the sheriff
when submitting?

[Ilya Sherman, 2016/11/14 17:57:32]

You should rename, yes.  Otherwise, the dashboard will show a confusing mix of
buckets under the same name.

     }                                                                       \
   }
 
 // Version of the above macro for cases which only care about time, not area.
 #define DEFINE_SCOPED_UMA_HISTOGRAM_TIMER(class_name, time_histogram)       \
   class class_name : public ScopedUMAHistogramAreaTimerBase {               \
    public:                                                                  \
     ~class_name();                                                          \
   };                                                                        \
   class_name::~class_name() {                                               \
@@ skipping 34 matching lines @@
   base::ElapsedTimer timer_;
   base::CheckedNumeric<int> area_;
 
   friend class ScopedUMAHistogramAreaTimerBaseTest;
   DISALLOW_COPY_AND_ASSIGN(ScopedUMAHistogramAreaTimerBase);
 };
 
 }  // namespace cc
 
 #endif  // CC_BASE_HISTOGRAMS_H_