Add StoragePartition's ProtocolHandlers at URLRequestContext construction time.

content/public/browser/browser_context.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_BROWSER_CONTEXT_H_
 #define CONTENT_PUBLIC_BROWSER_BROWSER_CONTEXT_H_
 
 #include "base/callback_forward.h"
 #include "base/hash_tables.h"
 #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;
 }
 
 namespace quota {
 class SpecialStoragePolicy;
 }
 
 class FilePath;
@@ skipping 66 matching lines @@
 
   // Return whether this context is incognito. Default is false.
   // This doesn't belong here; http://crbug.com/89628
   virtual bool IsOffTheRecord() const = 0;
 
   // Returns the request context information associated with this context.  Call
   // this only on the UI thread, since it can send notifications that should
   // 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
   // installed app's is_storage_isolated() returns false, this is equivalent to
   // calling GetRequestContext().
   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.
   // TODO(creis): Remove this version in favor of the one below.
   virtual net::URLRequestContextGetter* GetMediaRequestContext() = 0;
 
   // Returns the request context for media resources associated with this
   // context and renderer process.
   virtual net::URLRequestContextGetter* GetMediaRequestContextForRenderProcess(
       int renderer_child_id) = 0;
@@ skipping 32 matching lines @@
 struct hash<content::BrowserContext*> {
   std::size_t operator()(content::BrowserContext* const& p) const {
     return reinterpret_cast<std::size_t>(p);
   }
 };
 
 }  // namespace BASE_HASH_NAMESPACE
 #endif
 
 #endif  // CONTENT_PUBLIC_BROWSER_BROWSER_CONTEXT_H_