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

chrome/browser/media/webrtc_browsertest.cc

Index: chrome/browser/media/webrtc_browsertest.cc
diff --git a/chrome/browser/media/webrtc_browsertest.cc b/chrome/browser/media/webrtc_browsertest.cc
index 81e9033b46c206af256e943014eaa8a5c7391baa..2d873fff3e4394ce00cabeaccd4bdb50194d828e 100644
--- a/chrome/browser/media/webrtc_browsertest.cc
+++ b/chrome/browser/media/webrtc_browsertest.cc
@@ -56,7 +56,8 @@ class WebRtcBrowserTest : public WebRtcTestBase {
       const std::string& offer_cert_keygen_alg =
           WebRtcTestBase::kUseDefaultCertKeygen,
       const std::string& answer_cert_keygen_alg =
-          WebRtcTestBase::kUseDefaultCertKeygen) {
+          WebRtcTestBase::kUseDefaultCertKeygen,
+      bool get_stats = false) {

[phoglund_chromium, 2016/07/27 12:52:10]

Instead of adding a new parameter, maybe inline this function body to your new
test and just call stats verification directly from there. It's so few lines of
code it's not worth polluting this helper with more parameters.

[hbos_chromium, 2016/07/27 13:50:31]

Done.

     StartServerAndOpenTabs();
 
     SetupPeerconnectionWithLocalStream(left_tab_, offer_cert_keygen_alg);
@@ -64,6 +65,9 @@ class WebRtcBrowserTest : public WebRtcTestBase {
 
     NegotiateCall(left_tab_, right_tab_, video_codec);
 
+    if (get_stats)
+      GetStats(left_tab_);
+
     DetectVideoAndHangUp();
   }
 
@@ -192,3 +196,11 @@ IN_PROC_BROWSER_TEST_F(WebRtcBrowserTest,
                                     kKeygenAlgorithmEcdsa,
                                     kKeygenAlgorithmRsa);
 }
+
+IN_PROC_BROWSER_TEST_F(WebRtcBrowserTest,
+                       RunsAudioVideoWebRTCCallInTwoTabsGetStats) {
+  RunsAudioVideoWebRTCCallInTwoTabs(WebRtcTestBase::kUseDefaultVideoCodec,
+                                    WebRtcTestBase::kUseDefaultCertKeygen,
+                                    WebRtcTestBase::kUseDefaultCertKeygen,
+                                    true);
+}