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

content/renderer/media/rtc_peer_connection_handler_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/media/rtc_peer_connection_handler.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <string>
@@ skipping 55 matching lines @@
   message_loop->task_runner()->PostTask(FROM_HERE, quit_closure);
 }
 
 class MockRTCStatsResponse : public LocalRTCStatsResponse {
  public:
   MockRTCStatsResponse()
       : report_count_(0),
         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:
   int report_count_;
   int statistic_count_;
 };
 
 // Mocked wrapper for blink::WebRTCStatsRequest
 class MockRTCStatsRequest : public LocalRTCStatsRequest {
  public:
@@ skipping 963 matching lines @@
                                     testing::Ref(tracks[0])));
 
   std::unique_ptr<blink::WebRTCDTMFSenderHandler> sender(
       pc_handler_->createDTMFSender(tracks[0]));
   EXPECT_TRUE(sender.get());
 
   StopAllTracks(local_stream);
 }
 
 }  // namespace content