AppCache and StoragePartition'ing

content/browser/appcache/chrome_appcache_service.cc

Index: content/browser/appcache/chrome_appcache_service.cc
===================================================================
--- content/browser/appcache/chrome_appcache_service.cc	(revision 156155)
+++ content/browser/appcache/chrome_appcache_service.cc	(working copy)
@@ -9,6 +9,8 @@
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/browser/resource_context.h"
 #include "net/base/net_errors.h"
+#include "net/url_request/url_request_context_getter.h"
+#include "webkit/appcache/appcache_storage_impl.h"
 #include "webkit/quota/quota_manager.h"
 
 using content::BrowserThread;
@@ -22,13 +24,17 @@
 void ChromeAppCacheService::InitializeOnIOThread(
     const FilePath& cache_path,
     content::ResourceContext* resource_context,
+    net::URLRequestContextGetter* request_context_getter,
     scoped_refptr<quota::SpecialStoragePolicy> special_storage_policy) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   cache_path_ = cache_path;
   resource_context_ = resource_context;
-  set_request_context(resource_context->GetRequestContext());
 
+  // The |request_context_getter| can be NULL in some unit tests.

[awong, 2012/09/15 00:33:29]

Can you file a bug against me and add a TODO to remove this requirement once we
bring sanity to the malware tests?

[michaeln, 2012/09/15 01:50:56]

Done.

+  if (request_context_getter)
+    set_request_context(request_context_getter->GetURLRequestContext());
+
   // Init our base class.
   Initialize(
       cache_path_,