commit 26451933c1f41b4870d41ed92222cf3e726fb25f
author Harald Alvestrand <hta@webrtc.org> Thu Feb 21 10:27:15 2019
committer Commit Bot <commit-bot@chromium.org> Thu Feb 21 16:38:37 2019
tree 79151d99632a7c962bdff6b068112a410a30acea
parent ee95f3eacde5a5b1188ac5d2c35f5a6d55112047

DtlsTransport::ice_transport is const and can be called off thread

Since the ice_transport element of DtlsTransport is never changed
over the lifetime of the object, it's safe to return it on any thread.
Added const qualifier to make this obvious.

Bug: chromium:907849
Change-Id: Ib180682689a57df5106717b125c626d7ac9e7561
Reviewed-on: https://webrtc-review.googlesource.com/c/123781
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26800}

pc/dtls_transport.cc

diff --git a/pc/dtls_transport.cc b/pc/dtls_transport.cc
index 66e2787..38cef1e 100644
--- a/pc/dtls_transport.cc
+++ b/pc/dtls_transport.cc
@@ -45,12 +45,12 @@
     std::unique_ptr<cricket::DtlsTransportInternal> internal)
     : owner_thread_(rtc::Thread::Current()),
       info_(DtlsTransportState::kNew),
-      internal_dtls_transport_(std::move(internal)) {
+      internal_dtls_transport_(std::move(internal)),
+      ice_transport_(new rtc::RefCountedObject<IceTransportWithPointer>(
+          internal_dtls_transport_->ice_transport())) {
   RTC_DCHECK(internal_dtls_transport_.get());
   internal_dtls_transport_->SignalDtlsState.connect(
       this, &DtlsTransport::OnInternalDtlsState);
-  ice_transport_ = new rtc::RefCountedObject<IceTransportWithPointer>(
-      internal_dtls_transport_->ice_transport());
   UpdateInformation();
 }
 
@@ -77,8 +77,6 @@
 }
 
 rtc::scoped_refptr<IceTransportInterface> DtlsTransport::ice_transport() {
-  RTC_DCHECK_RUN_ON(owner_thread_);
-  rtc::CritScope scope(&lock_);
   return ice_transport_;
 }