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

jingle/glue/thread_wrapper.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "jingle/glue/thread_wrapper.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/bind.h"
@@ skipping 143 matching lines @@
   TRACE_EVENT2("webrtc", "JingleThreadWrapper::Dispatch", "src_file_and_line",
                message->posted_from.file_and_line(), "src_func",
                message->posted_from.function_name());
   message->phandler->OnMessage(message);
 }
 
 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();
   DCHECK(current_thread != nullptr) << "Send() can be called only from a "
       "thread that has JingleThreadWrapper.";
 
   rtc::Message message;
   message.posted_from = posted_from;
   message.phandler = handler;
   message.message_id = id;
@@ skipping 153 matching lines @@
 
 void JingleThreadWrapper::Stop() {
   NOTREACHED();
 }
 
 void JingleThreadWrapper::Run() {
   NOTREACHED();
 }
 
 }  // namespace jingle_glue