Support caching histograms so that a lookup isn't done in each iteration.

third_party/WebKit/Source/platform/Histogram.h

+// Copyright 2016 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 Histogram_h
+#define Histogram_h
+
+#include "platform/PlatformExport.h"
+#include <stdint.h>
+
+namespace base {
+class HistogramBase;
+};
+
+namespace blink {
+
+#define DEFINE_STATIC_LOCAL_HISTOGRAM(type, name, constructor) \
+    static type name constructor;

[esprehn, 2016/02/01 16:21:00]

please use DEFINE_STATIC_LOCAL, don't invent new macros

[dtapuska, 2016/02/01 16:28:51]

Sorry didn't know that actually existed.

+
+class PLATFORM_EXPORT CustomCountHistogram {
+public:
+    CustomCountHistogram(const char* name, int32_t min, int32_t max, int32_t bucketCount);
+    void count(int32_t sample);
+
+protected:
+    CustomCountHistogram(base::HistogramBase*);
+
+private:
+    base::HistogramBase* m_histogram;
+};
+
+class PLATFORM_EXPORT EnumerationHistogram : public CustomCountHistogram {
+public:
+    EnumerationHistogram(const char* name, int32_t boundaryValue);
+};
+
+class PLATFORM_EXPORT SparseHistogram {
+public:
+    SparseHistogram(const char* name);

[esprehn, 2016/02/01 16:21:00]

explicit

[dtapuska, 2016/02/01 16:28:51]

Done.

+
+    void sample(int32_t sample);
+
+private:
+    base::HistogramBase* m_histogram;
+};
+
+} // namespace blink
+
+#endif // Histogram_h