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

webkit/tools/test_shell/simple_resource_loader_bridge.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.
 //
 // This file contains an implementation of the ResourceLoaderBridge class.
 // The class is implemented using net::URLRequest, meaning it is a "simple"
 // version that directly issues requests. The more complicated one used in the
 // browser uses IPC.
 //
 // Because net::URLRequest only provides an asynchronous resource loading API,
@@ skipping 105 matching lines @@
 FileOverHTTPParams* g_file_over_http_params = NULL;
 
 //-----------------------------------------------------------------------------
 
 class IOThread : public base::Thread {
  public:
   IOThread() : base::Thread("IOThread") {
   }
 
   ~IOThread() {
-    // We cannot rely on our base class to stop the thread since we want our
-    // CleanUp function to run.
     Stop();
   }
 
   virtual void Init() {
     if (g_request_context_params) {
       g_request_context = new TestShellRequestContext(
           g_request_context_params->cache_path,
           g_request_context_params->cache_mode,
           g_request_context_params->no_proxy);
       delete g_request_context_params;
@@ skipping 904 matching lines @@
 
 // static
 void SimpleResourceLoaderBridge::AllowFileOverHTTP(
     const std::string& file_path_template, const GURL& http_prefix) {
   DCHECK(!file_path_template.empty());
   DCHECK(http_prefix.is_valid() &&
          (http_prefix.SchemeIs("http") || http_prefix.SchemeIs("https")));
   g_file_over_http_params = new FileOverHTTPParams(file_path_template,
                                                    http_prefix);
 }