Fix BrowserTest.ThirtyFourTabs and possibly other ui tests in single-process ...

chrome/browser/renderer_host/browser_render_process_host.cc

 // Copyright (c) 2006-2008 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.
 
 // Represents the browser side of the browser <--> renderer communication
 // channel. There will be one RenderProcessHost per renderer process.
 
 #include "chrome/browser/renderer_host/browser_render_process_host.h"
 
 #include "build/build_config.h"
@@ skipping 53 matching lines @@
 #include "chrome/common/temp_scaffolding_stubs.h"
 #endif
 
 #include "skia/include/SkBitmap.h"
 
 
 namespace {
 
 // ----------------------------------------------------------------------------
 
+// This class creates the IO thread for the renderer when running in
+// single-process mode.  It's not used in multi-process mode.
 class RendererMainThread : public base::Thread {
  public:
   explicit RendererMainThread(const std::wstring& channel_id)
       : base::Thread("Chrome_InProcRendererThread"),
-        channel_id_(channel_id) {
+        channel_id_(channel_id),
+        render_process_(NULL) {
   }
 
  protected:
   virtual void Init() {
 #if defined(OS_WIN)
     CoInitialize(NULL);
 #endif
 
-    render_process_.reset(new RenderProcess(channel_id_));
+    render_process_ = new RenderProcess(channel_id_);
     // It's a little lame to manually set this flag.  But the single process
     // RendererThread will receive the WM_QUIT.  We don't need to assert on
     // this thread, so just force the flag manually.
     // If we want to avoid this, we could create the InProcRendererThread
     // directly with _beginthreadex() rather than using the Thread class.
     base::Thread::SetThreadWasQuitProperly(true);
   }
 
   virtual void CleanUp() {
-    render_process_.reset();
+    delete render_process_;
 
 #if defined(OS_WIN)
     CoUninitialize();
 #endif
   }
 
  private:
   std::wstring channel_id_;
-  scoped_ptr<RenderProcess> render_process_;
+  // Deleted in CleanUp() on the renderer thread, so don't use a smart pointer.
+  RenderProcess* render_process_;
 };
 
 // Used for a View_ID where the renderer has not been attached yet
 const int32 kInvalidViewID = -1;
 
 // Get the path to the renderer executable, which is the same as the
 // current executable.
 bool GetRendererPath(std::wstring* cmd_line) {
   return PathService::Get(base::FILE_EXE, cmd_line);
 }
@@ skipping 758 matching lines @@
         SendUserScriptsUpdate(shared_memory);
       }
       break;
     }
     default: {
       NOTREACHED();
       break;
     }
   }
 }