Remove the Extensions URLRequestContext

content/browser/renderer_host/render_message_filter.h

Index: content/browser/renderer_host/render_message_filter.h
diff --git a/content/browser/renderer_host/render_message_filter.h b/content/browser/renderer_host/render_message_filter.h
index 4eb986471c936aeedabe1d0d33e83f5323415d89..7659a5a4dd4357ff2cc68ba3bb3bce22643ae1d7 100644
--- a/content/browser/renderer_host/render_message_filter.h
+++ b/content/browser/renderer_host/render_message_filter.h
@@ -60,12 +60,12 @@ struct MediaLogEvent;
 }
 
 namespace net {
-class URLRequestContext;
 class URLRequestContextGetter;
 }
 
 namespace content {
 class BrowserContext;
+class CookieStoreMap;
 class DOMStorageContextWrapper;
 class MediaInternals;
 class PluginServiceImpl;
@@ -85,6 +85,7 @@ class RenderMessageFilter : public BrowserMessageFilter {
                       PluginServiceImpl * plugin_service,
                       BrowserContext* browser_context,
                       net::URLRequestContextGetter* request_context,
+                      const CookieStoreMap& cookie_store_map,
                       RenderWidgetHelper* render_widget_helper,
                       media::AudioManager* audio_manager,
                       MediaInternals* media_internals,
@@ -105,11 +106,6 @@ class RenderMessageFilter : public BrowserMessageFilter {
 
   int render_process_id() const { return render_process_id_; }
 
-  // Returns the correct net::URLRequestContext depending on what type of url is
-  // given.
-  // Only call on the IO thread.
-  net::URLRequestContext* GetRequestContextForURL(const GURL& url);
-
  private:
   friend class BrowserThread;
   friend class base::DeleteHelper<RenderMessageFilter>;
@@ -272,6 +268,10 @@ class RenderMessageFilter : public BrowserMessageFilter {
   // Contextual information to be used for requests created here.
   scoped_refptr<net::URLRequestContextGetter> request_context_;
 
+

[Charlie Reis, 2013/08/17 00:17:22]

nit: Remove extra blank line.

[awong, 2013/08/17 00:32:52]

Done.

+  // Map of schemes to the CookieStore instance that services those schemes.

[Charlie Reis, 2013/08/17 00:17:22]

Maybe mention why we keep a clone of it here, rather than the same one as the
StoragePartition.  (There's an assumption that it won't change after the time of
the clone, right?)

[awong, 2013/08/17 00:32:52]

Added comment about thread safety and assumption on immutability.

+  scoped_ptr<CookieStoreMap> cookie_store_map_;
+
   // The ResourceContext which is to be used on the IO thread.
   ResourceContext* resource_context_;