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

chrome_frame/test/urlmon_moniker_integration_test.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 <atlbase.h>
 #include <atlcom.h>
 
 #include "base/threading/thread.h"
 #include "base/win/scoped_comptr.h"
 #include "base/win/scoped_handle.h"
@@ skipping 41 matching lines @@
 
 // Runs an HTTP server on a worker thread that has a message loop.
 class RunTestServer : public base::Thread {
  public:
   RunTestServer()
       : base::Thread("TestServer"),
       default_response_("/", kTestContent),
       ready_(::CreateEvent(NULL, TRUE, FALSE, NULL)) {
   }
 
+  ~RunTestServer() {
+    Stop();
+  }
+
   bool Start() {
     bool ret = StartWithOptions(Options(MessageLoop::TYPE_UI, 0));
     if (ret) {
       message_loop()->PostTask(FROM_HERE,
           NewRunnableFunction(&RunTestServer::StartServer, this));
       wait_until_ready();
     }
     return ret;
   }
 
@@ skipping 267 matching lines @@
     callback2.SetCallbackExpectations(
         UrlmonMonikerTestCallback::EXPECT_NO_CALL, S_OK, false);
     hr = callback2.CreateUrlMonikerAndBindToStorage(test_url, NULL);
     // S_OK means that the operation completed synchronously.
     // Otherwise we'd get MK_S_ASYNCHRONOUS.
     EXPECT_EQ(S_OK, hr);
   }
 }
 
 */