Add a blob field to ServiceWorkerFetchResponse and read the blob

content/browser/storage_partition_impl_map.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/browser/storage_partition_impl_map.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/file_util.h"
 #include "base/files/file_enumerator.h"
@@ skipping 397 matching lines @@
                 partition->GetAppCacheService(),
                 blob_storage_context));
   }
   protocol_handlers[kChromeDevToolsScheme] =
       linked_ptr<net::URLRequestJobFactory::ProtocolHandler>(
           CreateDevToolsProtocolHandler(browser_context_->GetResourceContext(),
                                         browser_context_->IsOffTheRecord()));
 
   ProtocolHandlerScopedVector protocol_interceptors;
   protocol_interceptors.push_back(
-      ServiceWorkerRequestHandler::CreateInterceptor().release());
+      ServiceWorkerRequestHandler::CreateInterceptor(blob_storage_context)

[michaeln, 2014/05/22 03:26:22]

A lot of plumbing to get this value from here to where its needed and adds as
coded adds an odd bit of state to this otherwise stateless class. A different
approach might eliminate some of the plumbing (maybe) and put it next to other
similar plumbing (definitely).

ServiceWorkerRequestHandler::InitializeHandler(
   p1, sw_context_wrapper,...,
   filter_->blob_storage_context());

That method is how we associate the swcontext with a requesthandler which is
very similar in nature.

+          .release());
 
   // These calls must happen after StoragePartitionImpl::Create().
   if (partition_domain.empty()) {
     partition->SetURLRequestContext(
         GetContentClient()->browser()->CreateRequestContext(
             browser_context_,
             &protocol_handlers,
             protocol_interceptors.Pass()));
   } else {
     partition->SetURLRequestContext(
@@ skipping 132 matching lines @@
 
     // We do not call InitializeURLRequestContext() for media contexts because,
     // other than the HTTP cache, the media contexts share the same backing
     // objects as their associated "normal" request context.  Thus, the previous
     // call serves to initialize the media request context for this storage
     // partition as well.
   }
 }
 
 }  // namespace content