commit 448c387b8274c4416378e51cda29d135613c4c22
author Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> Wed Feb 13 09:17:50 2019
committer Commit Bot <commit-bot@chromium.org> Wed Feb 13 12:45:12 2019
tree 71ee22659423a25e6dc21f216c95d5b8aba69cd4
parent 2bd54a1bd9c485b009b93b9bc936d525396044a1

IceTransportWithTransportChannel: Initialize |thread_checker_| in declaration

This works around https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89305, which
causes GCC to fail to build the code due to |thread_checker_| being const
there and not having a declared constructor.

    ../../api/ice_transport_factory.cc: In constructor ‘webrtc::{anonymous}::IceTransportWithTransportChannel::IceTransportWithTransportChannel(std::unique_ptr<cricket::IceTransportInternal>)’:
    ../../api/ice_transport_factory.cc:31:3: error: uninitialized const member in ‘const class rtc::ThreadChecker’ [-fpermissive]
       IceTransportWithTransportChannel(
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    ../../api/ice_transport_factory.cc:45:28: note: ‘const rtc::ThreadChecker webrtc::{anonymous}::IceTransportWithTransportChannel::thread_checker_’ should be initialized
       const rtc::ThreadChecker thread_checker_;
                                ^~~~~~~~~~~~~~~

Bug: chromium:819294
Change-Id: I750e8cdd796b3b0e076de01194cf7de988ac4ce2
Reviewed-on: https://webrtc-review.googlesource.com/c/122820
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Raphael Kubo da Costa (rakuco) <raphael.kubo.da.costa@intel.com>
Cr-Commit-Position: refs/heads/master@{#26662}

api/ice_transport_factory.cc

diff --git a/api/ice_transport_factory.cc b/api/ice_transport_factory.cc
index b632f09..6cc399a 100644
--- a/api/ice_transport_factory.cc
+++ b/api/ice_transport_factory.cc
@@ -42,7 +42,7 @@
   }
 
  private:
-  const rtc::ThreadChecker thread_checker_;
+  const rtc::ThreadChecker thread_checker_{};
   const std::unique_ptr<cricket::IceTransportInternal> internal_
       RTC_GUARDED_BY(thread_checker_);
 };