Migrate part of the BackgroundFetchJobController to the UI thread

content/browser/background_fetch/background_fetch_context.cc

 // Copyright 2017 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/background_fetch/background_fetch_context.h"
 
 #include "base/memory/ptr_util.h"
 #include "content/browser/background_fetch/background_fetch_data_manager.h"
 #include "content/browser/background_fetch/background_fetch_job_controller.h"
 #include "content/browser/background_fetch/background_fetch_registration_id.h"
 #include "content/browser/service_worker/service_worker_context_wrapper.h"
 #include "content/browser/storage_partition_impl.h"
 #include "content/public/browser/browser_context.h"
+#include "net/url_request/url_request_context_getter.h"
 #include "url/origin.h"
 
 namespace content {
 
 BackgroundFetchContext::BackgroundFetchContext(
     BrowserContext* browser_context,
     StoragePartitionImpl* storage_partition,
     const scoped_refptr<ServiceWorkerContextWrapper>& service_worker_context)
     : browser_context_(browser_context),
-      storage_partition_(storage_partition),
       service_worker_context_(service_worker_context),
       background_fetch_data_manager_(
           base::MakeUnique<BackgroundFetchDataManager>(browser_context)) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   download_manager_ = BrowserContext::GetDownloadManager(browser_context_);
+  request_context_ =
+      make_scoped_refptr(storage_partition->GetURLRequestContext());
 }
 
 BackgroundFetchContext::~BackgroundFetchContext() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 }
 
 void BackgroundFetchContext::Shutdown() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
@@ skipping 78 matching lines @@
 
   return controller;
 }
 
 void BackgroundFetchContext::CreateController(
     const BackgroundFetchRegistrationId& registration_id,
     const BackgroundFetchOptions& options,
     std::vector<BackgroundFetchRequestInfo> initial_requests) {
   std::unique_ptr<BackgroundFetchJobController> controller =
       base::MakeUnique<BackgroundFetchJobController>(
-          registration_id, options, download_manager_, storage_partition_,
-          background_fetch_data_manager_.get(),
+          registration_id, options, background_fetch_data_manager_.get(),
+          download_manager_, request_context_,
           base::BindOnce(&BackgroundFetchContext::DidCompleteJob, this));
 
-  // TODO(peter): Start actually fetching the files.
+  // TODO(peter): We should actually be able to use Background Fetch in layout
+  // tests. That requires a download manager and a request context.
+  if (download_manager_ && request_context_) {
+    // Start fetching the |initial_requests| immediately. At some point in the
+    // future we may want a more elaborate scheduling mechanism here.
+    controller->Start(std::move(initial_requests));
+  }
 
   active_fetches_.insert(
       std::make_pair(registration_id, std::move(controller)));
 }
 
 void BackgroundFetchContext::DidCompleteJob(
     BackgroundFetchJobController* controller) {
   const BackgroundFetchRegistrationId& registration_id =
       controller->registration_id();
 
   DCHECK_GT(active_fetches_.count(registration_id), 0u);
 
   if (controller->state() == BackgroundFetchJobController::State::COMPLETED) {
     // TODO(peter): Dispatch the `backgroundfetched` or `backgroundfetchfail`
     // event to the Service Worker to inform the developer.
   }
 
   active_fetches_.erase(registration_id);
 }
 
 }  // namespace content