Remove direct usage of BrowserThreadImpl in tests

content/browser/download/base_file_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 "content/browser/download/base_file.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <utility>
 
 #include "base/files/file.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/logging.h"
 #include "base/macros.h"
-#include "base/message_loop/message_loop.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/test/test_file_util.h"
 #include "build/build_config.h"
-#include "content/browser/browser_thread_impl.h"
 #include "content/public/browser/download_interrupt_reasons.h"
+#include "content/public/test/test_browser_thread_bundle.h"
 #include "crypto/secure_hash.h"
 #include "crypto/sha2.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace content {
 namespace {
 
 const char kTestData1[] = "Let's write some data to the file!\n";
 const char kTestData2[] = "Writing more data.\n";
 const char kTestData3[] = "Final line.";
@@ skipping 17 matching lines @@
 
 }  // namespace
 
 class BaseFileTest : public testing::Test {
  public:
   static const unsigned char kEmptySha256Hash[crypto::kSHA256Length];
 
   BaseFileTest()
       : expect_file_survives_(false),
         expect_in_progress_(true),
-        expected_error_(DOWNLOAD_INTERRUPT_REASON_NONE),
-        file_thread_(BrowserThread::FILE, &message_loop_) {
-  }
+        expected_error_(DOWNLOAD_INTERRUPT_REASON_NONE) {}
 
   void SetUp() override {
     ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
     base_file_.reset(new BaseFile(net::NetLogWithSource()));
   }
 
   void TearDown() override {
     EXPECT_FALSE(base_file_->in_progress());
     if (!expected_error_) {
       EXPECT_EQ(static_cast<int64_t>(expected_data_.size()),
@@ skipping 120 matching lines @@
   bool expect_file_survives_;
 
   // Expect the file to be in progress.
   bool expect_in_progress_;
 
  private:
   // Keep track of what data should be saved to the disk file.
   std::string expected_data_;
   DownloadInterruptReason expected_error_;
 
-  // Mock file thread to satisfy debug checks in BaseFile.
-  base::MessageLoop message_loop_;
-  BrowserThreadImpl file_thread_;
+  TestBrowserThreadBundle thread_bundle_;
 };
 
 // This will initialize the entire array to zero.
 const unsigned char BaseFileTest::kEmptySha256Hash[] = { 0 };
 
 // Test the most basic scenario: just create the object and do a sanity check
 // on all its accessors. This is actually a case that rarely happens
 // in production, where we would at least Initialize it.
 TEST_F(BaseFileTest, CreateDestroy) {
   EXPECT_EQ(base::FilePath().value(), base_file_->full_path().value());
@@ skipping 512 matching lines @@
 
   const char kData[] = "hello";
   const int kDataLength = static_cast<int>(arraysize(kData) - 1);
   ASSERT_EQ(kDataLength, base::WriteFile(full_path, kData, kDataLength));
   // The file that we created here should stick around when the BaseFile is
   // destroyed during TearDown.
   expect_file_survives_ = true;
 }
 
 }  // namespace content