 Chromium Code Reviews
 Chromium Code Reviews Issue 2863123002:
  Wire up BWE stats through WebrtcSession so that they are filled in both for audio and video calls.  (Closed)
    
  
    Issue 2863123002:
  Wire up BWE stats through WebrtcSession so that they are filled in both for audio and video calls.  (Closed) 
  | Index: webrtc/pc/channel.cc | 
| diff --git a/webrtc/pc/channel.cc b/webrtc/pc/channel.cc | 
| index f5428a43effac7a93fce554e11946c220946337c..d0025079e41082f04d8119f64a9ad3f13afff97b 100644 | 
| --- a/webrtc/pc/channel.cc | 
| +++ b/webrtc/pc/channel.cc | 
| @@ -1987,6 +1987,12 @@ void VideoChannel::UpdateMediaSendRecvState_w() { | 
| LOG(LS_INFO) << "Changing video state, send=" << send; | 
| } | 
| +void VideoChannel::FillBitrateInfo(BandwidthEstimationInfo* bwe_info) { | 
| + RTC_DCHECK(InvokeOnWorker( | 
| 
Taylor Brandstetter
2017/05/07 21:30:44
Why is this in an RTC_DCHECK?
 
stefan-webrtc
2017/05/08 07:12:56
Copy paste from below, but my method didn't have t
 | 
| + RTC_FROM_HERE, | 
| + Bind(&VideoMediaChannel::FillBitrateInfo, media_channel(), bwe_info))); | 
| +} | 
| + | 
| bool VideoChannel::GetStats(VideoMediaInfo* stats) { | 
| return InvokeOnWorker(RTC_FROM_HERE, Bind(&VideoMediaChannel::GetStats, | 
| media_channel(), stats)); |