Allow the content browser client to specify a special cookie store to be

content/browser/loader/resource_dispatcher_host_impl.cc

Index: content/browser/loader/resource_dispatcher_host_impl.cc
===================================================================
--- content/browser/loader/resource_dispatcher_host_impl.cc	(revision 255909)
+++ content/browser/loader/resource_dispatcher_host_impl.cc	(working copy)
@@ -1057,9 +1057,12 @@
   }
 
   // Construct the request.
+  net::CookieStore* cookie_store =
+      GetContentClient()->browser()->OverrideCookieStoreForRenderProcess(

[mmenke, 2014/03/12 16:43:00]

I think "OverrideCookieStoreForRenderProcess" isn't a good name for this -
function names should generally be verbs, and read as a verb phrase,
"OverrideCookieStoreForRenderProcess" is a bit misleading.

Maybe add a "Get" in front?

Also, from a prerender perspective, is this enough information to be sure it's a
prerender?  There's nothing to stop other render views from being created in a
process with a prerender in it.

[tburkard, 2014/03/12 16:51:32]

jam suggested this name.

Also, if you look at the .h file containing the definition, you can see eg
OverrideRequestContextForURL, which is analogous to this and would also need a
"Get" based on your reasoning.

The Get is implied by the return type.

+          child_id);
   scoped_ptr<net::URLRequest> new_request;
   new_request = request_context->CreateRequest(
-      request_data.url, request_data.priority, NULL);
+      request_data.url, request_data.priority, NULL, cookie_store);
 
   new_request->set_method(request_data.method);
   new_request->set_first_party_for_cookies(
@@ -1360,8 +1363,13 @@
     return;
   }
 
+  net::CookieStore* cookie_store =
+      GetContentClient()->browser()->OverrideCookieStoreForRenderProcess(
+          child_id);
   scoped_ptr<net::URLRequest> request(
-      request_context->CreateRequest(url, net::DEFAULT_PRIORITY, NULL));
+      request_context->CreateRequest(url, net::DEFAULT_PRIORITY, NULL,
+                                     cookie_store));
+
   request->set_method("GET");
   SetReferrerForRequest(request.get(), referrer);