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

chrome_frame/utils.h

 // 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.
 
 #ifndef CHROME_FRAME_UTILS_H_
 #define CHROME_FRAME_UTILS_H_
 
 #include <OAidl.h>
 #include <objidl.h>
 #include <windows.h>
@@ skipping 369 matching lines @@
         QueryInterfaceIfDelegateSupports<_ComMapClass>)
 
 // Queries the delegated COM object for an interface, bypassing the wrapper.
 #define COM_INTERFACE_BLIND_DELEGATE() \
     COM_INTERFACE_ENTRY_FUNC_BLIND(0, CheckOutgoingInterface<_ComMapClass>)
 
 // Thread that enters STA and has a UI message loop.
 class STAThread : public base::Thread {
  public:
   explicit STAThread(const char *name) : Thread(name) {}
+  ~STAThread() {
+    Stop();
+  }
   bool Start() {
     return StartWithOptions(Options(MessageLoop::TYPE_UI, 0));
   }
  protected:
   // Called just prior to starting the message loop
   virtual void Init() {
     ::CoInitialize(0);
   }
 
   // Called just after the message loop ends
@@ skipping 200 matching lines @@
 bool IsChromeFrameDocument(IWebBrowser2* web_browser);
 
 // Increases the wininet connection limit for HTTP 1.0/1.1 connections to the
 // value passed in. This is only done if the existing connection limit is
 // lesser than the connection limit passed in. This function attempts to
 // increase the connection count once per process.
 // Returns true on success.
 bool IncreaseWinInetConnections(DWORD connections);
 
 #endif  // CHROME_FRAME_UTILS_H_