Add a default implementation in metrics_default.cc of histograms methods in system_wrappers/interfac

webrtc/system_wrappers/include/metrics_default.h

Index: webrtc/system_wrappers/include/metrics_default.h
diff --git a/webrtc/system_wrappers/include/metrics_default.h b/webrtc/system_wrappers/include/metrics_default.h
new file mode 100644
index 0000000000000000000000000000000000000000..fe7165688a262bf4d9db4798bd969cfd84abe3da
--- /dev/null
+++ b/webrtc/system_wrappers/include/metrics_default.h
@@ -0,0 +1,57 @@
+/*
+ *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
+ *
+ *  Use of this source code is governed by a BSD-style license
+ *  that can be found in the LICENSE file in the root of the source
+ *  tree. An additional intellectual property rights grant can be found
+ *  in the file PATENTS.  All contributing project authors may
+ *  be found in the AUTHORS file in the root of the source tree.
+ */
+
+#ifndef WEBRTC_SYSTEM_WRAPPERS_INCLUDE_METRICS_DEFAULT_H_
+#define WEBRTC_SYSTEM_WRAPPERS_INCLUDE_METRICS_DEFAULT_H_
+
+#include <map>
+#include <memory>
+#include <string>
+
+namespace webrtc {
+namespace metrics {
+
+struct SampleInfo {
+  SampleInfo(const std::string& name, int min, int max, int bucket_count);

[pbos-webrtc, 2016/05/06 19:58:39]

size_t bucket_count

[åsapersson, 2016/05/09 14:47:41]

Done.

+  ~SampleInfo();
+
+  const std::string name;
+  const int min;
+  const int max;
+  const int bucket_count;

[pbos-webrtc, 2016/05/06 19:58:40]

size_t

[åsapersson, 2016/05/09 14:47:40]

Done.

+  std::map<int, int> samples;  // <value, # of events>

[pbos-webrtc, 2016/05/06 19:58:39]

Shouldn't this be a std::vector of ints that's initialized to the size of
bucket_count? That way we can do a fast lookup without samples.find(), but we
need to map |sample| to bucket_index.

[åsapersson, 2016/05/09 14:47:40]

Since most metrics are an aggregate over the call and not logged frequently,
this might not be needed right now? I added a todo that we should consider
changing this...

sample.find() is only used in test functionality now, right?

+};
+
+// Enables collection of samples.
+// This method should be called before any other call into webrtc.
+void Enable();
+
+// Gets histograms and clears all samples.
+void GetAndReset(
+    std::map<std::string, std::unique_ptr<SampleInfo>>* histograms);
+
+// Functions below are mainly for testing.
+
+// Clears all samples.
+void Reset();
+
+// Returns the number of times the |sample| has been added to the histogram.
+int NumEvents(const std::string& name, int sample);

[pbos-webrtc, 2016/05/06 19:58:39]

If we do go with respecting buckets, I think we should rename this
NumEventsInBucket or something like that. Same below.

+
+// Returns the total number of added samples to the histogram.
+int NumSamples(const std::string& name);
+
+// Returns the minimum sample value (or -1 if the histogram has no samples).
+int MinSample(const std::string& name);
+
+}  // namespace metrics
+}  // namespace webrtc
+
+#endif  // WEBRTC_SYSTEM_WRAPPERS_INCLUDE_METRICS_DEFAULT_H_