Chromium Code Reviews| Index: webrtc/api/fakemetricsobserver.cc |
| diff --git a/webrtc/api/fakemetricsobserver.cc b/webrtc/api/fakemetricsobserver.cc |
| index 71e71b3432bacf719c88065cc9da8e72346e4596..1b6265d7d5676f9ad8b61a0e85cbcd24257bb9f8 100644 |
| --- a/webrtc/api/fakemetricsobserver.cc |
| +++ b/webrtc/api/fakemetricsobserver.cc |
| @@ -45,7 +45,9 @@ void FakeMetricsObserver::AddHistogramSample(PeerConnectionMetricsName type, |
| int FakeMetricsObserver::GetEnumCounter(PeerConnectionEnumCounterType type, |
| int counter) const { |
| RTC_DCHECK(thread_checker_.CalledOnValidThread()); |
| - RTC_CHECK(counters_.size() > static_cast<size_t>(type)); |
| + if (counters_.size() <= static_cast<size_t>(type)) { |
| + return 0; |
| + } |
|
Taylor Brandstetter
2016/10/03 20:33:34
What's the purpose of this change?
honghaiz3
2016/10/04 01:03:54
If nothing was recorded, it should return 0, not a
|
| const auto& it = counters_[type].find(counter); |
| if (it == counters_[type].end()) { |
| return 0; |