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

 // Copyright (c) 2009 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/browser/renderer_host/test/test_render_view_host.h"
 
 #include "base/gfx/rect.h"
 #include "chrome/browser/renderer_host/backing_store.h"
 #include "chrome/browser/tab_contents/test_tab_contents.h"
 #include "chrome/common/render_messages.h"
@@ skipping 86 matching lines @@
 
 void RenderViewHostTestHarness::SetUp() {
   // See comment above profile_ decl for why we check for NULL here.
   if (!profile_.get())
     profile_.reset(new TestingProfile());
 
   // This will be deleted when the TabContents goes away.
   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() {
   contents_.reset();
 
   // Make sure that we flush any messages related to TabContents destruction
   // before we destroy the profile.
   MessageLoop::current()->RunAllPending();
 }