Do not access member fStop_ variable webrtc::Thread class directly.

jingle/glue/thread_wrapper.cc

Index: jingle/glue/thread_wrapper.cc
diff --git a/jingle/glue/thread_wrapper.cc b/jingle/glue/thread_wrapper.cc
index f49c6e9cd745d79a9f7fba857bc57f86c2e23c0c..c3001829ee915a9a4f598a8d3dbb868ed398b8fc 100644
--- a/jingle/glue/thread_wrapper.cc
+++ b/jingle/glue/thread_wrapper.cc
@@ -161,7 +161,7 @@ void JingleThreadWrapper::Send(const rtc::Location& posted_from,
                                rtc::MessageHandler* handler,
                                uint32_t id,
                                rtc::MessageData* data) {
-  if (fStop_)
+  if (IsQuitting())

[Nicolas Zea, 2016/07/18 17:40:31]

IsQuitting seems to be overriden below to just NOTREACHED(), no?

[Sergey Ulanov, 2016/07/18 19:53:06]

I think you can just remove these these two lines. fStop_ is always false here.

     return;
 
   JingleThreadWrapper* current_thread = JingleThreadWrapper::current();