Add instrumentation to track the IceEndpointType.

talk/app/webrtc/fakemetricsobserver.cc

 /*
  * libjingle
  * Copyright 2015 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
  *
  *  1. Redistributions of source code must retain the above copyright notice,
  *     this list of conditions and the following disclaimer.
  *  2. Redistributions in binary form must reproduce the above copyright notice,
@@ skipping 18 matching lines @@
 #include "webrtc/base/checks.h"
 
 namespace webrtc {
 
 FakeMetricsObserver::FakeMetricsObserver() {
   Reset();
 }
 
 void FakeMetricsObserver::Reset() {
   DCHECK(thread_checker_.CalledOnValidThread());
-  memset(counters_, 0, sizeof(counters_));
+  counters_ = std::vector<std::vector<int>>();

[juberti1, 2015/08/20 23:14:25]

this could be done via counters_.clear(), I think

   memset(int_histogram_samples_, 0, sizeof(int_histogram_samples_));
   for (std::string& type : string_histogram_samples_) {
     type.clear();
   }
 }
 
-void FakeMetricsObserver::IncrementCounter(PeerConnectionMetricsCounter type) {
+void FakeMetricsObserver::IncrementEnumCounter(
+    PeerConnectionEnumCounterType type,
+    int counter,
+    int counter_max) {

[juberti1, 2015/08/20 23:14:25]

I don't understand why we need to pass in a maximum here. It seems unnecessary
and forces us to add Max entries to enums that don't really need them.

   DCHECK(thread_checker_.CalledOnValidThread());
-  ++counters_[type];
+  if (counters_.size() <= static_cast<size_t>(type)) {
+    counters_.resize(type + 1);
+  }
+  auto& counters = counters_[type];
+  if (counters.size() < static_cast<size_t>(counter_max)) {
+    counters.resize(counter_max);
+  }
+  ++counters[counter];
 }
 
 void FakeMetricsObserver::AddHistogramSample(PeerConnectionMetricsName type,
     int value) {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK_EQ(int_histogram_samples_[type], 0);
   int_histogram_samples_[type] = value;
 }
 
 void FakeMetricsObserver::AddHistogramSample(PeerConnectionMetricsName type,
     const std::string& value) {
   DCHECK(thread_checker_.CalledOnValidThread());
   string_histogram_samples_[type].assign(value);
 }
 
-int FakeMetricsObserver::GetCounter(PeerConnectionMetricsCounter type) const {
+int FakeMetricsObserver::GetEnumCounter(PeerConnectionEnumCounterType type,
+                                        int counter) const {
   DCHECK(thread_checker_.CalledOnValidThread());
-  return counters_[type];
+  CHECK(counters_.size() > static_cast<size_t>(type) &&
+        counters_[type].size() > static_cast<size_t>(counter));
+  return counters_[type][counter];
 }
 
 int FakeMetricsObserver::GetIntHistogramSample(
     PeerConnectionMetricsName type) const {
   DCHECK(thread_checker_.CalledOnValidThread());
   return int_histogram_samples_[type];
 }
 
 const std::string& FakeMetricsObserver::GetStringHistogramSample(
     PeerConnectionMetricsName type) const {
   DCHECK(thread_checker_.CalledOnValidThread());
   return string_histogram_samples_[type];
 }
 
 }  // namespace webrtc