Add TestBrowserThreadBundle into RenderViewHostTestHarness. Kill some unnecessary real threads.

chrome/browser/browsing_data/browsing_data_appcache_helper_unittest.cc

 // Copyright (c) 2012 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/browsing_data/browsing_data_appcache_helper.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/stl_util.h"
 #include "chrome/test/base/testing_profile.h"
+#include "content/public/test/test_browser_thread_bundle.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 class TestCompletionCallback {
  public:
   TestCompletionCallback()
       : have_result_(false) {
   }
 
   bool have_result() const { return have_result_; }
 
   void callback() {
     have_result_ = true;
   }
 
  private:
   bool have_result_;
 };
 
 }  // namespace
 
-typedef testing::Test CannedBrowsingDataAppCacheHelperTest;
+class CannedBrowsingDataAppCacheHelperTest : public testing::Test  {
+ private:
+  content::TestBrowserThreadBundle thread_bundle_;

[jam, 2013/05/31 17:29:25]

why does this need it now?

[awong, 2013/05/31 20:57:17]

Leaked in from the CL I split out out of. Next CL will need it, but I've removed
it for now.

+};
 
 TEST_F(CannedBrowsingDataAppCacheHelperTest, SetInfo) {
   TestingProfile profile;
 
   GURL manifest1("http://example1.com/manifest.xml");
   GURL manifest2("http://example2.com/path1/manifest.xml");
   GURL manifest3("http://example2.com/path2/manifest.xml");
 
   scoped_refptr<CannedBrowsingDataAppCacheHelper> helper(
       new CannedBrowsingDataAppCacheHelper(&profile));
@@ skipping 70 matching lines @@
 
   scoped_refptr<CannedBrowsingDataAppCacheHelper> helper(
       new CannedBrowsingDataAppCacheHelper(&profile));
 
   ASSERT_TRUE(helper->empty());
   helper->AddAppCache(manifest1);
   ASSERT_TRUE(helper->empty());
   helper->AddAppCache(manifest2);
   ASSERT_TRUE(helper->empty());
 }