Added ServiceWorkerResponses to GetJobResponse callback

content/browser/background_fetch/background_fetch_data_manager.cc

Index: content/browser/background_fetch/background_fetch_data_manager.cc
diff --git a/content/browser/background_fetch/background_fetch_data_manager.cc b/content/browser/background_fetch/background_fetch_data_manager.cc
index 3a5661ce7a580307292f4da64045e66309b57498..c199fc7f75ed8c15d263196d3db6217aa810c081 100644
--- a/content/browser/background_fetch/background_fetch_data_manager.cc
+++ b/content/browser/background_fetch/background_fetch_data_manager.cc
@@ -8,6 +8,7 @@
 #include "content/browser/background_fetch/background_fetch_context.h"
 #include "content/browser/background_fetch/background_fetch_job_response_data.h"
 #include "content/browser/background_fetch/background_fetch_request_info.h"
+#include "content/browser/blob_storage/chrome_blob_storage_context.h"
 #include "content/public/browser/blob_handle.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/download_interrupt_reasons.h"
@@ -97,6 +98,17 @@ BackgroundFetchDataManager::GetRequestInfo(const std::string& job_guid,
 void BackgroundFetchDataManager::GetJobResponse(
     const std::string& job_guid,
     const BackgroundFetchResponseCompleteCallback& callback) {
+  BrowserThread::PostTaskAndReplyWithResult(
+      BrowserThread::IO, FROM_HERE,

[Peter Beverloo, 2017/03/27 11:05:16]

BrowserThread::UI

This class lives on the IO thread, ChromeBlobStorageContext::GetFor() must be
invoked on the UI thread.

[harkness, 2017/03/27 12:23:19]

According to the comments, only the constructor should be called on the UI
thread. It is odd that it does a post task to the IO thread, but it doesn't seem
to require that it be called on any particular thread.

[harkness, 2017/03/27 12:35:32]

Or maybe the comments are wrong!  :)

+      base::Bind(&ChromeBlobStorageContext::GetFor, browser_context_),
+      base::Bind(&BackgroundFetchDataManager::DidGetBlobStorageContext,
+                 weak_ptr_factory_.GetWeakPtr(), job_guid, callback));
+}
+
+void BackgroundFetchDataManager::DidGetBlobStorageContext(
+    const std::string& job_guid,
+    const BackgroundFetchResponseCompleteCallback& callback,
+    ChromeBlobStorageContext* blob_context) {

[Peter Beverloo, 2017/03/27 11:05:16]

DCHECK(blob_context);

[harkness, 2017/03/27 12:23:19]

Done.

   BackgroundFetchJobInfo* job_info = job_map_[job_guid].get();
   DCHECK(job_info);
 
@@ -105,6 +117,9 @@ void BackgroundFetchDataManager::GetJobResponse(
   job_info->set_job_response_data(
       base::MakeUnique<BackgroundFetchJobResponseData>(job_info->num_requests(),
                                                        callback));
+  BackgroundFetchJobResponseData* job_response_data =
+      job_info->job_response_data();
+  DCHECK(job_response_data);
 
   // Iterate over the requests and create blobs for each response.
   for (size_t request_index = 0; request_index < job_info->num_requests();
@@ -115,28 +130,14 @@ void BackgroundFetchDataManager::GetJobResponse(
 
     // TODO(harkness): Only create a blob response if the request was
     // successful. Otherwise create an error response.
-    content::BrowserContext::CreateFileBackedBlob(
-        browser_context_, request_info->file_path(), 0 /* offset */,
-        request_info->received_bytes(),
-        base::Time() /* expected_modification_time */,
-        base::Bind(&BackgroundFetchDataManager::DidGetRequestResponse,
-                   weak_ptr_factory_.GetWeakPtr(), job_guid, request_index));
-  }
-}
+    std::unique_ptr<BlobHandle> blob_handle =
+        blob_context->CreateFileBackedBlob(
+            request_info->file_path(), 0 /* offset */,
+            request_info->received_bytes(),
+            base::Time() /* expected_modification_time */);
 
-void BackgroundFetchDataManager::DidGetRequestResponse(
-    const std::string& job_guid,
-    int request_sequence_number,
-    std::unique_ptr<BlobHandle> blob_handle) {
-  BackgroundFetchJobInfo* job_info = job_map_[job_guid].get();
-  DCHECK(job_info);
-
-  BackgroundFetchJobResponseData* job_response_data =
-      job_info->job_response_data();
-  DCHECK(job_response_data);
-
-  job_response_data->AddResponse(request_sequence_number,
-                                 std::move(blob_handle));
+    job_response_data->AddResponse(*request_info.get(), std::move(blob_handle));
+  }
 }
 
 bool BackgroundFetchDataManager::UpdateRequestState(