First of several patches to get rid of MessageLoop caching now that we have C...

chrome/browser/renderer_host/test/test_render_view_host.cc

Index: chrome/browser/renderer_host/test/test_render_view_host.cc
===================================================================
--- chrome/browser/renderer_host/test/test_render_view_host.cc	(revision 30289)
+++ chrome/browser/renderer_host/test/test_render_view_host.cc	(working copy)
@@ -104,6 +104,8 @@
   SiteInstance* instance = SiteInstance::CreateSiteInstance(profile_.get());
 
   contents_.reset(new TestTabContents(profile_.get(), instance));
+
+  user_data_manager_.reset(UserDataManager::Create());

[tony, 2009/10/28 20:27:33]

Is this supposed to be in this change?

[jam, 2009/10/28 20:31:08]

Yeah, this is needed because of the change to new_tab_ui.cc, specifically taking
out "if (g_browser_process->io_thread()) {".  Now we create NewTabHTMLSource
which internally uses the UserDataManager.  This occurs in a bunch of tests that
use RenderViewHostTestHarness.

 }
 
 void RenderViewHostTestHarness::TearDown() {