Fixed tsan warnings in remoting_unittests

remoting/jingle_glue/jingle_thread.cc

 // Copyright (c) 2010 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 "remoting/jingle_glue/jingle_thread.h"
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/message_pump.h"
 #include "base/time.h"
@@ skipping 74 matching lines @@
 
   task_pump_ = NULL;
   message_loop_ = NULL;
 }
 
 void JingleThread::Stop() {
   // Shutdown gracefully: make sure that we excute all messages left in the
   // queue before exiting. Thread::Stop() would not do that.
   Post(this, kStopMessageId);
   stopped_event_.Wait();
+
+  // This will wait until the thread is actually finished.
+  Thread::Stop();
 }
 
 MessageLoop* JingleThread::message_loop() {
   return message_loop_;
 }
 
   // Returns task pump if the thread is running, otherwise NULL is returned.
 TaskPump* JingleThread::task_pump() {
   return task_pump_;
 }
@@ skipping 15 matching lines @@
     // otherwise post another task to try again later.
     if (msgq_.size() > 0 || fPeekKeep_) {
       Post(this, kStopMessageId);
     } else {
       MessageQueue::Quit();
     }
   }
 }
 
 }  // namespace remoting