[ServiceWorker] Sends the blob uuid of the request body to the ServiceWorker.

content/browser/service_worker/service_worker_controllee_request_handler.cc

Index: content/browser/service_worker/service_worker_controllee_request_handler.cc
diff --git a/content/browser/service_worker/service_worker_controllee_request_handler.cc b/content/browser/service_worker/service_worker_controllee_request_handler.cc
index bd8205b4e90a435b6418b52e71018d116b305e41..c37946e2c0dfa06301690ce5d76b2b4384bbdcb8 100644
--- a/content/browser/service_worker/service_worker_controllee_request_handler.cc
+++ b/content/browser/service_worker/service_worker_controllee_request_handler.cc
@@ -10,6 +10,7 @@
 #include "content/browser/service_worker/service_worker_registration.h"
 #include "content/browser/service_worker/service_worker_url_request_job.h"
 #include "content/browser/service_worker/service_worker_utils.h"
+#include "content/common/resource_request_body.h"
 #include "content/common/service_worker/service_worker_types.h"
 #include "net/base/load_flags.h"
 #include "net/base/net_util.h"
@@ -21,13 +22,15 @@ ServiceWorkerControlleeRequestHandler::ServiceWorkerControlleeRequestHandler(
     base::WeakPtr<ServiceWorkerContextCore> context,
     base::WeakPtr<ServiceWorkerProviderHost> provider_host,
     base::WeakPtr<webkit_blob::BlobStorageContext> blob_storage_context,
-    ResourceType resource_type)
+    ResourceType resource_type,
+    scoped_refptr<ResourceRequestBody> body)
     : ServiceWorkerRequestHandler(context,
                                   provider_host,
                                   blob_storage_context,
                                   resource_type),
       is_main_resource_load_(
           ServiceWorkerUtils::IsMainResourceType(resource_type)),
+      body_(body),

[michaeln1, 2014/08/19 22:03:25]

It's pretty hidden that the body has a collection of
webkit_blob::BlobDataHandles attached to it using the userdata mechanism. Those
handles ensure the blobs don't get freed up for as long as the
ResourceRequestBody instance is around. ResolveBlobReference in the
UploadDataStreamBuilder sets that up.

It might be worth a comment. Also might be worth hoisting the SetUserData calls
up from the UploadDataStreamBuilder to somewhere closer to IPC message
deserialization (maybe RDH::BeginRequest) since there are two consumers
depending on that behavior now.

[horo, 2014/08/20 02:28:02]

Added comments in service_worker_url_request_job.h.
I will create another cl to move SetUserData calls.

       weak_factory_(this) {
 }
 
@@ -79,7 +82,7 @@ net::URLRequestJob* ServiceWorkerControlleeRequestHandler::MaybeCreateJob(
   DCHECK(!job_.get() || job_->ShouldForwardToServiceWorker());
 
   job_ = new ServiceWorkerURLRequestJob(
-      request, network_delegate, provider_host_, blob_storage_context_);
+      request, network_delegate, provider_host_, blob_storage_context_, body_);
   if (is_main_resource_load_)
     PrepareForMainResource(request->url());
   else