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

chrome/service/service_process.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 "chrome/service/service_process.h"
 
 #include <algorithm>
 
 #include "base/basictypes.h"
 #include "base/command_line.h"
@@ skipping 44 matching lines @@
 
  protected:
   virtual void CleanUp();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ServiceIOThread);
 };
 
 ServiceIOThread::ServiceIOThread(const char* name) : base::Thread(name) {}
 ServiceIOThread::~ServiceIOThread() {
-  // We cannot rely on our base class to stop the thread since we want our
-  // CleanUp function to run.
   Stop();
 }
 
 void ServiceIOThread::CleanUp() {
   content::URLFetcher::CancelAll();
 }
 
 // Prepares the localized strings that are going to be displayed to
 // the user if the service process dies. These strings are stored in the
 // environment block so they are accessible in the early stages of the
@@ skipping 285 matching lines @@
     } else {
       Shutdown();
     }
   }
 }
 
 ServiceProcess::~ServiceProcess() {
   Teardown();
   g_service_process = NULL;
 }