Add TestBrowserThreadBundle into RenderViewHostTestHarness. Kill some unnecessary real threads.

chrome/browser/safe_browsing/browser_feature_extractor_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/safe_browsing/browser_feature_extractor.h"
 
 #include <map>
 #include <string>
 #include <vector>
 
@@ skipping 14 matching lines @@
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/page_transition_types.h"
 #include "content/public/common/referrer.h"
 #include "content/public/test/test_browser_thread.h"
 #include "content/public/test/web_contents_tester.h"
 #include "googleurl/src/gurl.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
-using ::testing::Return;
-using ::testing::StrictMock;
-using content::BrowserThread;
 using content::WebContentsTester;
+using testing::Return;
+using testing::StrictMock;
 
 namespace safe_browsing {
 namespace {
 class MockClientSideDetectionService : public ClientSideDetectionService {
  public:
   MockClientSideDetectionService() : ClientSideDetectionService(NULL) {}
   virtual ~MockClientSideDetectionService() {};
 
   MOCK_CONST_METHOD1(IsBadIpAddress, bool(const std::string&));
 };
 }  // namespace
 
 class BrowserFeatureExtractorTest : public ChromeRenderViewHostTestHarness {
  protected:
-  BrowserFeatureExtractorTest()
-      : ui_thread_(BrowserThread::UI, &message_loop_) {
-  }
-
   virtual void SetUp() {
     ChromeRenderViewHostTestHarness::SetUp();
     profile()->CreateHistoryService(true /* delete_file */, false /* no_db */);
     service_.reset(new StrictMock<MockClientSideDetectionService>());
     extractor_.reset(
         new BrowserFeatureExtractor(web_contents(), service_.get()));
     num_pending_ = 0;
     browse_info_.reset(new BrowseInfo);
   }
 
@@ skipping 87 matching lines @@
           request.feature_map(i);
       EXPECT_EQ(0U, features->count(feature.name()));
       std::set<std::string> meta_infos;
       for (int j = 0; j < feature.metainfo_size(); ++j) {
         meta_infos.insert(feature.metainfo(j));
       }
       (*features)[feature.name()] = meta_infos;
     }
   }
 
-  content::TestBrowserThread ui_thread_;
   int num_pending_;
   scoped_ptr<BrowserFeatureExtractor> extractor_;
   std::map<ClientPhishingRequest*, bool> success_;
   scoped_ptr<BrowseInfo> browse_info_;
   scoped_ptr<MockClientSideDetectionService> service_;
 
  private:
   void ExtractFeaturesDone(bool success, ClientPhishingRequest* request) {
     ASSERT_EQ(0U, success_.count(request));
     success_[request] = success;
@@ skipping 400 matching lines @@
   EXPECT_TRUE(hosts.find("evil.com") != hosts.end());
   feature_name = base::StringPrintf("%s%s", features::kBadIpFetch,
                                     "92.92.92.92");
   EXPECT_TRUE(features.count(feature_name));
   hosts = features[feature_name];
   EXPECT_EQ(2U, hosts.size());
   EXPECT_TRUE(hosts.find("bad.com") != hosts.end());
   EXPECT_TRUE(hosts.find("evil.com") != hosts.end());
 }
 }  // namespace safe_browsing