Downloads: replace BrowserThread::FILE with task scheduler.

content/public/test/test_file_error_injector.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/public/test/test_file_error_injector.h"
 
 #include <utility>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "content/browser/download/download_file_factory.h"
 #include "content/browser/download/download_file_impl.h"
 #include "content/browser/download/download_interrupt_reasons_impl.h"
 #include "content/browser/download/download_manager_impl.h"
+#include "content/browser/download/download_task_runner.h"
 #include "content/browser/loader/resource_dispatcher_host_impl.h"
 #include "content/public/browser/browser_thread.h"
 #include "url/gurl.h"
 
 namespace net {
 class NetLogWithSource;
 }
 
 namespace content {
 class ByteStreamReader;
@@ skipping 86 matching lines @@
     const TestFileErrorInjector::FileErrorInfo& error_info,
     const base::Closure& ctor_callback,
     const base::Closure& dtor_callback)
     : DownloadFileImpl(std::move(save_info),
                        default_download_directory,
                        std::move(byte_stream),
                        net_log,
                        observer),
       error_info_(error_info),
       destruction_callback_(dtor_callback) {
-  // DownloadFiles are created on the UI thread and are destroyed on the FILE
-  // thread. Schedule the ConstructionCallback on the FILE thread so that if a
-  // DownloadItem schedules a DownloadFile to be destroyed and creates another
-  // one (as happens during download resumption), then the DestructionCallback
-  // for the old DownloadFile is run before the ConstructionCallback for the
-  // next DownloadFile.
-  BrowserThread::PostTask(BrowserThread::FILE, FROM_HERE, ctor_callback);
+  // DownloadFiles are created on the UI thread and are destroyed on the
+  // download task runner. Schedule the ConstructionCallback on the
+  // download task runner, so that if a DownloadItem schedules a
+  // DownloadFile to be destroyed and creates another one (as happens during
+  // download resumption), then the DestructionCallback for the old DownloadFile
+  // is run before the ConstructionCallback for the next DownloadFile.
+  GetDownloadTaskRunner()->PostTask(FROM_HERE, ctor_callback);
 }
 
 DownloadFileWithError::~DownloadFileWithError() {
-  DCHECK_CURRENTLY_ON(BrowserThread::FILE);
+  DCHECK(GetDownloadTaskRunner()->RunsTasksInCurrentSequence());
   destruction_callback_.Run();
 }
 
 void DownloadFileWithError::Initialize(
     const InitializeCallback& initialize_callback,
     const CancelRequestCallback& cancel_request_callback,
     const DownloadItem::ReceivedSlices& received_slices,
     bool is_parallelizable) {
   DownloadInterruptReason error_to_return = DOWNLOAD_INTERRUPT_REASON_NONE;
   InitializeCallback callback_to_use = initialize_callback;
@@ skipping 148 matching lines @@
 }
 
 DownloadFileWithErrorFactory::~DownloadFileWithErrorFactory() {}
 
 DownloadFile* DownloadFileWithErrorFactory::CreateFile(
     std::unique_ptr<DownloadSaveInfo> save_info,
     const base::FilePath& default_download_directory,
     std::unique_ptr<ByteStreamReader> byte_stream,
     const net::NetLogWithSource& net_log,
     base::WeakPtr<DownloadDestinationObserver> observer) {
-  return new DownloadFileWithError(std::move(save_info),
-                                   default_download_directory,
-                                   std::move(byte_stream),
-                                   net_log,
-                                   observer,
-                                   injected_error_,
-                                   construction_callback_,
-                                   destruction_callback_);
+  return new DownloadFileWithError(
+      std::move(save_info), default_download_directory, std::move(byte_stream),
+      net_log, observer, injected_error_, construction_callback_,
+      destruction_callback_);
 }
 
 bool DownloadFileWithErrorFactory::SetError(
     TestFileErrorInjector::FileErrorInfo error) {
   injected_error_ = std::move(error);
   return true;
 }
 
 TestFileErrorInjector::TestFileErrorInjector(DownloadManager* download_manager)
     :  // This code is only used for browser_tests, so a
@@ skipping 94 matching lines @@
     case FILE_OPERATION_RENAME_ANNOTATE:
       return "RENAME_ANNOTATE";
     default:
       break;
   }
 
   return "Unknown";
 }
 
 }  // namespace content