WebRTCStats added, retaining type info when surfacing WebRTC stats into Blink

content/renderer/media/rtc_peer_connection_handler_unittest.cc

Index: content/renderer/media/rtc_peer_connection_handler_unittest.cc
diff --git a/content/renderer/media/rtc_peer_connection_handler_unittest.cc b/content/renderer/media/rtc_peer_connection_handler_unittest.cc
index 1032489c6219280751ee5de2cc54828f254b7f0d..b008b1816cef06eea17b22adaea1751e927b3260 100644
--- a/content/renderer/media/rtc_peer_connection_handler_unittest.cc
+++ b/content/renderer/media/rtc_peer_connection_handler_unittest.cc
@@ -73,18 +73,14 @@ class MockRTCStatsResponse : public LocalRTCStatsResponse {
         statistic_count_(0) {
   }
 
-  size_t addReport(blink::WebString type,
-                   blink::WebString id,
-                   double timestamp) override {
+  void addStats(const blink::WebRTCStats& stats) override {
     ++report_count_;
-    return report_count_;
+    for (std::unique_ptr<blink::WebRTCStatsMemberIterator> member(
+          stats.iterator()); !member->isEnd(); member->next()) {

[tommi (sloooow) - chröme, 2016/07/27 18:38:40]

is this what git cl format did?

[hbos_chromium, 2016/07/28 08:27:45]

No. Done. I ran git cl format now and it put a newline after ...iterator());

+      ++statistic_count_;
+    }
   }
 
-  void addStatistic(size_t report,
-                    blink::WebString name,
-                    blink::WebString value) override {
-    ++statistic_count_;
-  }
   int report_count() const { return report_count_; }
 
  private: