admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Uncomment thread-checkers in ChannelProxy

Browse Source 
The thread-check should be pass unit-tests, following https://codereview.webrtc.org/2998923002/.

BUG=webrtc:7405
TBR=stefan@webrtc.org

Review-Url: https://codereview.webrtc.org/2995243002
Cr-Commit-Position: refs/heads/master@{#19510}
This commit is contained in:
eladalon 2017-08-25 01:50:58 -07:00 committed by Commit Bot
parent 151aa6b3f7
commit 27e812e369
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
webrtc/voice_engine/channel_proxy.cc
View File

@ -319,17 +319,13 @@ bool ChannelProxy::GetRecCodec(CodecInst* codec_inst) const {
}
void ChannelProxy::OnTwccBasedUplinkPacketLossRate(float packet_loss_rate) {
// TODO(eladalon): This fails in UT; fix and uncomment.
// See: https://bugs.chromium.org/p/webrtc/issues/detail?id=7405
// RTC_DCHECK(worker_thread_checker_.CalledOnValidThread());
RTC_DCHECK(worker_thread_checker_.CalledOnValidThread());
channel()->OnTwccBasedUplinkPacketLossRate(packet_loss_rate);
}
void ChannelProxy::OnRecoverableUplinkPacketLossRate(
float recoverable_packet_loss_rate) {
// TODO(eladalon): This fails in UT; fix and uncomment.
// See: https://bugs.chromium.org/p/webrtc/issues/detail?id=7405
// RTC_DCHECK(worker_thread_checker_.CalledOnValidThread());
RTC_DCHECK(worker_thread_checker_.CalledOnValidThread());
channel()->OnRecoverableUplinkPacketLossRate(recoverable_packet_loss_rate);
}