Thread::Stop() must be called before any subclass's destructor completes.

dbus/test_service.cc

 // Copyright (c) 2011 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 "dbus/test_service.h"
 
 #include "base/bind.h"
 #include "base/test/test_timeouts.h"
 #include "base/threading/platform_thread.h"
 #include "dbus/bus.h"
@@ skipping 12 matching lines @@
 }
 
 TestService::TestService(const Options& options)
     : base::Thread("TestService"),
       dbus_thread_message_loop_proxy_(options.dbus_thread_message_loop_proxy),
       on_all_methods_exported_(false, false),
       num_exported_methods_(0) {
 }
 
 TestService::~TestService() {
+  Stop();
 }
 
 bool TestService::StartService() {
   base::Thread::Options thread_options;
   thread_options.message_loop_type = MessageLoop::TYPE_IO;
   return StartWithOptions(thread_options);
 }
 
 bool TestService::WaitUntilServiceIsStarted() {
   const base::TimeDelta timeout(
@@ skipping 134 matching lines @@
 Response* TestService::SlowEcho(MethodCall* method_call) {
   base::PlatformThread::Sleep(TestTimeouts::tiny_timeout_ms());
   return Echo(method_call);
 }
 
 Response* TestService::BrokenMethod(MethodCall* method_call) {
   return NULL;
 }
 
 }  // namespace dbus