Wire up BWE stats through WebrtcSession so that they are filled in both for audio and video calls.

webrtc/pc/webrtcsession.cc

Index: webrtc/pc/webrtcsession.cc
diff --git a/webrtc/pc/webrtcsession.cc b/webrtc/pc/webrtcsession.cc
index cc424b222e081c994b6a7e546f59c126d86a7c01..a7f97885f02c959cee39ce9165fdc228354888d2 100644
--- a/webrtc/pc/webrtcsession.cc
+++ b/webrtc/pc/webrtcsession.cc
@@ -1895,6 +1895,10 @@ bool WebRtcSession::CreateDataChannel(const cricket::ContentInfo* content,
   return true;
 }
 
+Call::Stats WebRtcSession::GetCallStats() {
+  return call_->GetStats();

[hbos, 2017/05/09 12:48:14]

I suspect this is not safe? call_ is a raw pointer, typically accessed by
call_w() which DCHECKs to be on the worker thread and might modify it. Can you
add a DCHECK or invoke on worker?

[holmer, 2017/05/30 14:44:29]

As discussed offline, I have no added thread checkers and introduced a
GetCallStats_s() method to be used in statscollector.cc.

+}
+
 std::unique_ptr<SessionStats> WebRtcSession::GetStats_n(
     const ChannelNamePairs& channel_name_pairs) {
   RTC_DCHECK(network_thread()->IsCurrent());