commit 1d02d3e5e68e5025ddf8ab3af11e416778cd0201
author asapersson <asapersson@webrtc.org> Fri Sep 09 22:40:25 2016 -0700
committer Commit bot <commit-bot@chromium.org> Sat Sep 10 05:40:34 2016 +0000
tree 453b118ffe31f4ede860d023c8fa87feb9e6e688
parent d5fff5040c691b02486c407c41616c6009483bf1

Remove RTC_LOGGED_* macro.

BUG=

Review-Url: https://codereview.webrtc.org/2326843003
Cr-Commit-Position: refs/heads/master@{#14174}

webrtc/system_wrappers/include/metrics.h

diff --git a/webrtc/system_wrappers/include/metrics.h b/webrtc/system_wrappers/include/metrics.h
index 0fee344..ceddc95 100644
--- a/webrtc/system_wrappers/include/metrics.h
+++ b/webrtc/system_wrappers/include/metrics.h
@@ -82,30 +82,7 @@
   RTC_HISTOGRAM_COUNTS(name, sample, 1, 100000, 50)
 
 #define RTC_HISTOGRAM_COUNTS(name, sample, min, max, bucket_count) \
-  RTC_HISTOGRAM_COMMON_BLOCK(name, sample, false, \
-      webrtc::metrics::HistogramFactoryGetCounts(name, min, max, bucket_count))
-
-// RTC_HISTOGRAM_COUNTS with logging.
-#define RTC_LOGGED_HISTOGRAM_COUNTS_100(name, sample) \
-  RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 100, 50)
-
-#define RTC_LOGGED_HISTOGRAM_COUNTS_200(name, sample) \
-  RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 200, 50)
-
-#define RTC_LOGGED_HISTOGRAM_COUNTS_500(name, sample) \
-  RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 500, 50)
-
-#define RTC_LOGGED_HISTOGRAM_COUNTS_1000(name, sample) \
-  RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 1000, 50)
-
-#define RTC_LOGGED_HISTOGRAM_COUNTS_10000(name, sample) \
-  RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 10000, 50)
-
-#define RTC_LOGGED_HISTOGRAM_COUNTS_100000(name, sample) \
-  RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 100000, 50)
-
-#define RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, min, max, bucket_count) \
-  RTC_HISTOGRAM_COMMON_BLOCK(name, sample, true, \
+  RTC_HISTOGRAM_COMMON_BLOCK(name, sample, \
       webrtc::metrics::HistogramFactoryGetCounts(name, min, max, bucket_count))
 
 // Deprecated.
@@ -121,37 +98,21 @@
 #define RTC_HISTOGRAM_PERCENTAGE(name, sample) \
   RTC_HISTOGRAM_ENUMERATION(name, sample, 101)
 
-// RTC_HISTOGRAM_PERCENTAGE with logging.
-#define RTC_LOGGED_HISTOGRAM_PERCENTAGE(name, sample) \
-  RTC_LOGGED_HISTOGRAM_ENUMERATION(name, sample, 101)
-
 // Histogram for booleans.
 #define RTC_HISTOGRAM_BOOLEAN(name, sample) \
   RTC_HISTOGRAM_ENUMERATION(name, sample, 2)
 
-// RTC_HISTOGRAM_BOOLEAN with logging.
-#define RTC_LOGGED_HISTOGRAM_BOOLEAN(name, sample) \
-  RTC_LOGGED_HISTOGRAM_ENUMERATION(name, sample, 2)
-
 // Histogram for enumerators (evenly spaced buckets).
 // |boundary| should be above the max enumerator sample.
 #define RTC_HISTOGRAM_ENUMERATION(name, sample, boundary) \
-  RTC_HISTOGRAM_COMMON_BLOCK(name, sample, false, \
-      webrtc::metrics::HistogramFactoryGetEnumeration(name, boundary))
-
-// RTC_HISTOGRAM_ENUMERATION with logging.
-#define RTC_LOGGED_HISTOGRAM_ENUMERATION(name, sample, boundary) \
-  RTC_HISTOGRAM_COMMON_BLOCK(name, sample, true, \
+  RTC_HISTOGRAM_COMMON_BLOCK(name, sample, \
       webrtc::metrics::HistogramFactoryGetEnumeration(name, boundary))
 
 // The name of the histogram should not vary.
 // TODO(asapersson): Consider changing string to const char*.
-#define RTC_HISTOGRAM_COMMON_BLOCK(constant_name, sample, log, \
+#define RTC_HISTOGRAM_COMMON_BLOCK(constant_name, sample, \
                                    factory_get_invocation) \
   do { \
-    if (log) { \
-      LOG(LS_INFO) << constant_name << " " << sample; \
-    } \
     static webrtc::metrics::Histogram* atomic_histogram_pointer = nullptr; \
     webrtc::metrics::Histogram* histogram_pointer = \
         rtc::AtomicOps::AcquireLoadPtr(&atomic_histogram_pointer); \
@@ -213,39 +174,6 @@
   RTC_HISTOGRAMS_COMMON(index, name, sample, \
       RTC_HISTOGRAM_PERCENTAGE(name, sample))
 
-// RTC_HISTOGRAMS_COUNTS with logging.
-#define RTC_LOGGED_HISTOGRAMS_COUNTS_100(index, name, sample) \
-  RTC_HISTOGRAMS_COMMON(index, name, sample, \
-      RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 100, 50))
-
-#define RTC_LOGGED_HISTOGRAMS_COUNTS_200(index, name, sample) \
-  RTC_HISTOGRAMS_COMMON(index, name, sample, \
-      RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 200, 50))
-
-#define RTC_LOGGED_HISTOGRAMS_COUNTS_500(index, name, sample) \
-  RTC_HISTOGRAMS_COMMON(index, name, sample, \
-      RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 500, 50))
-
-#define RTC_LOGGED_HISTOGRAMS_COUNTS_1000(index, name, sample) \
-  RTC_HISTOGRAMS_COMMON(index, name, sample, \
-      RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 1000, 50))
-
-#define RTC_LOGGED_HISTOGRAMS_COUNTS_10000(index, name, sample) \
-  RTC_HISTOGRAMS_COMMON(index, name, sample, \
-      RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 10000, 50))
-
-#define RTC_LOGGED_HISTOGRAMS_COUNTS_100000(index, name, sample) \
-  RTC_HISTOGRAMS_COMMON(index, name, sample, \
-      RTC_LOGGED_HISTOGRAM_COUNTS(name, sample, 1, 100000, 50))
-
-#define RTC_LOGGED_HISTOGRAMS_ENUMERATION(index, name, sample, boundary) \
-  RTC_HISTOGRAMS_COMMON(index, name, sample, \
-      RTC_LOGGED_HISTOGRAM_ENUMERATION(name, sample, boundary))
-
-#define RTC_LOGGED_HISTOGRAMS_PERCENTAGE(index, name, sample) \
-  RTC_HISTOGRAMS_COMMON(index, name, sample, \
-      RTC_LOGGED_HISTOGRAM_PERCENTAGE(name, sample))
-
 #define RTC_HISTOGRAMS_COMMON(index, name, sample, macro_invocation) \
   do { \
     switch (index) { \