From 179f99730775f1de20f7dd178f7784b0f64df1d9 Mon Sep 17 00:00:00 2001 From: zstein Date: Tue, 13 Jun 2017 15:01:49 -0700 Subject: [PATCH] Remove DCHECK from PeerConnectionFactory::worker_thread. PeerConnection::SetBitrate calls PeerConnectionFactory::worker_thread from multiple threads, so it was triggering the DCHECK. However, the worker thread never changes after construction, so worker_thread should be safe to call from multiple threads. BUG=NONE Review-Url: https://codereview.webrtc.org/2923953004 Cr-Commit-Position: refs/heads/master@{#18576} --- webrtc/pc/peerconnectionfactory.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/webrtc/pc/peerconnectionfactory.cc b/webrtc/pc/peerconnectionfactory.cc index 777bac1eb5..77e6f2c272 100644 --- a/webrtc/pc/peerconnectionfactory.cc +++ b/webrtc/pc/peerconnectionfactory.cc @@ -374,7 +374,6 @@ rtc::Thread* PeerConnectionFactory::signaling_thread() { } rtc::Thread* PeerConnectionFactory::worker_thread() { - RTC_DCHECK(signaling_thread_->IsCurrent()); return worker_thread_; }