Chromium Code Reviews| Index: talk/app/webrtc/fakemetricsobserver.h |
| diff --git a/talk/app/webrtc/fakemetricsobserver.h b/talk/app/webrtc/fakemetricsobserver.h |
| index 39454bf4db48862a32b0b7d7b64ed6a1bc184502..3e0f10f25b06a2410d435318017e82c1312e42b7 100644 |
| --- a/talk/app/webrtc/fakemetricsobserver.h |
| +++ b/talk/app/webrtc/fakemetricsobserver.h |
| @@ -46,14 +46,10 @@ class FakeMetricsObserver : public MetricsObserverInterface { |
| int counter_max) override; |
| void AddHistogramSample(PeerConnectionMetricsName type, |
| int value) override; |
| - void AddHistogramSample(PeerConnectionMetricsName type, |
| - const std::string& value) override; |
| // Accessors to be used by the tests. |
| int GetEnumCounter(PeerConnectionEnumCounterType type, int counter) const; |
| - int GetIntHistogramSample(PeerConnectionMetricsName type) const; |
| - const std::string& GetStringHistogramSample( |
| - PeerConnectionMetricsName type) const; |
| + int GetHistogramSample(PeerConnectionMetricsName type) const; |
| protected: |
| ~FakeMetricsObserver() {} |
| @@ -62,9 +58,8 @@ class FakeMetricsObserver : public MetricsObserverInterface { |
| rtc::ThreadChecker thread_checker_; |
| // This is a 2 dimension array. The first index is the enum counter type. The |
|
juberti
2015/09/24 13:41:15
This comment could use some clarification. It's no
guoweis_webrtc
2015/09/24 18:27:13
Done.
|
| // 2nd index is the counter of that particular enum counter type. |
| - std::vector<std::vector<int>> counters_; |
| - int int_histogram_samples_[kPeerConnectionMetricsName_Max]; |
| - std::string string_histogram_samples_[kPeerConnectionMetricsName_Max]; |
| + std::vector<std::map<int, int>> counters_; |
| + int histogram_samples_[kPeerConnectionMetricsName_Max]; |
| }; |
| } // namespace webrtc |