Some minor fixes for Background Fetch

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_event_dispatcher.h"
 #include "content/browser/background_fetch/background_fetch_job_controller.h"
@@ skipping 26 matching lines @@
 BackgroundFetchContext::BackgroundFetchContext(
     BrowserContext* browser_context,
     StoragePartitionImpl* storage_partition,
     scoped_refptr<ServiceWorkerContextWrapper> service_worker_context)
     : browser_context_(browser_context),
       data_manager_(
           base::MakeUnique<BackgroundFetchDataManager>(browser_context)),
       event_dispatcher_(base::MakeUnique<BackgroundFetchEventDispatcher>(
           std::move(service_worker_context))) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
-  request_context_ =
-      make_scoped_refptr(storage_partition->GetURLRequestContext());
 }
 
 BackgroundFetchContext::~BackgroundFetchContext() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 }
 
+void BackgroundFetchContext::InitializeOnIOThread(
+    scoped_refptr<net::URLRequestContextGetter> request_context_getter) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  request_context_getter_ = request_context_getter;
+}
+
 void BackgroundFetchContext::Shutdown() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&BackgroundFetchContext::ShutdownOnIO, this));
 }
 
 void BackgroundFetchContext::ShutdownOnIO() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   active_fetches_.clear();
@@ skipping 74 matching lines @@
   return controller;
 }
 
 void BackgroundFetchContext::CreateController(
     const BackgroundFetchRegistrationId& registration_id,
     const BackgroundFetchOptions& options,
     std::vector<scoped_refptr<BackgroundFetchRequestInfo>> initial_requests) {
   std::unique_ptr<BackgroundFetchJobController> controller =
       base::MakeUnique<BackgroundFetchJobController>(
           registration_id, options, data_manager_.get(), browser_context_,
-          request_context_,
+          request_context_getter_,
           base::BindOnce(&BackgroundFetchContext::DidCompleteJob, this));
 
   // TODO(peter): We should actually be able to use Background Fetch in layout
   // tests. That requires a download manager and a request context.
-  if (request_context_) {
+  if (request_context_getter_) {
     // 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(
@@ skipping 48 matching lines @@
 
   // Delete all persistent information associated with the |registration_id|.
   data_manager_->DeleteRegistration(
       registration_id, base::BindOnce(&RecordRegistrationDeletedError));
 
   // Delete the local state associated with the |registration_id|.
   active_fetches_.erase(registration_id);
 }
 
 }  // namespace content