Add StoragePartition's ProtocolHandlers at URLRequestContext construction time.

content/public/browser/browser_context.h

Index: content/public/browser/browser_context.h
diff --git a/content/public/browser/browser_context.h b/content/public/browser/browser_context.h
index f2837f7aad2783b73b86ca74252ac74173d707df..4860a1b7bf32cd59942a6cd365edfb9d71628e40 100644
--- a/content/public/browser/browser_context.h
+++ b/content/public/browser/browser_context.h
@@ -10,6 +10,7 @@
 #include "base/memory/scoped_ptr.h"
 #include "base/supports_user_data.h"
 #include "content/common/content_export.h"
+#include "net/url_request/url_request_job_factory.h"
 
 namespace net {
 class URLRequestContextGetter;
@@ -96,6 +97,13 @@ class CONTENT_EXPORT BrowserContext : public base::SupportsUserData {
   // happen on the UI thread.
   // TODO(creis): Remove this version in favor of the one below.
   virtual net::URLRequestContextGetter* GetRequestContext() = 0;
+  virtual net::URLRequestContextGetter* CreateRequestContext(

[awong, 2012/12/13 01:06:15]

Can you add a commenting to the CreateXXX functions saying this should only be
called once per BrowserContext object and then add a TODO(ajwong) to remove
these once we've resolved http://159193?

Also, I wonder if these would be better put in ContentBrowserClient.

If you think about it, this is really a hole that allows the embedder to specify
how to create a particular type of URLRequestContext...

[pauljensen, 2012/12/13 17:58:44]

Done.

+      scoped_ptr<net::URLRequestJobFactory::ProtocolHandler>
+          blob_protocol_handler,
+      scoped_ptr<net::URLRequestJobFactory::ProtocolHandler>
+          file_system_protocol_handler,
+      scoped_ptr<net::URLRequestJobFactory::Interceptor>
+          developer_protocol_handler) = 0;
 
   // Returns the request context appropriate for the given renderer. If the
   // renderer process doesn't have an associated installed app, or if the
@@ -104,9 +112,15 @@ class CONTENT_EXPORT BrowserContext : public base::SupportsUserData {
   virtual net::URLRequestContextGetter* GetRequestContextForRenderProcess(
       int renderer_child_id) = 0;
 
-  virtual net::URLRequestContextGetter* GetRequestContextForStoragePartition(
+  virtual net::URLRequestContextGetter* CreateRequestContextForStoragePartition(

[awong, 2012/12/13 01:06:15]

Same comment as previoiusly.

[pauljensen, 2012/12/13 17:58:44]

Done.

       const FilePath& partition_path,
-      bool in_memory) = 0;
+      bool in_memory,
+      scoped_ptr<net::URLRequestJobFactory::ProtocolHandler>
+          blob_protocol_handler,
+      scoped_ptr<net::URLRequestJobFactory::ProtocolHandler>
+          file_system_protocol_handler,
+      scoped_ptr<net::URLRequestJobFactory::Interceptor>
+          developer_protocol_handler) = 0;
 
   // Returns the default request context for media resources associated with
   // this context.