From 27e812e36927e15401259e6a1a2cd01b9fdd4f79 Mon Sep 17 00:00:00 2001 From: eladalon Date: Fri, 25 Aug 2017 01:50:58 -0700 Subject: [PATCH] Uncomment thread-checkers in ChannelProxy 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} --- webrtc/voice_engine/channel_proxy.cc | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/webrtc/voice_engine/channel_proxy.cc b/webrtc/voice_engine/channel_proxy.cc index 027659d2f0..cdd4b6624f 100644 --- a/webrtc/voice_engine/channel_proxy.cc +++ b/webrtc/voice_engine/channel_proxy.cc @@ -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); }