| Index: content/browser/renderer_host/render_message_filter.cc
|
| diff --git a/content/browser/renderer_host/render_message_filter.cc b/content/browser/renderer_host/render_message_filter.cc
|
| index 3f2f100b8499855513647252b14a970f008b2de1..d06c8f948d1b57c433e6a8286aa5c501970e9d7e 100644
|
| --- a/content/browser/renderer_host/render_message_filter.cc
|
| +++ b/content/browser/renderer_host/render_message_filter.cc
|
| @@ -38,6 +38,7 @@
|
| #include "content/public/browser/browser_context.h"
|
| #include "content/public/browser/browser_thread.h"
|
| #include "content/public/browser/content_browser_client.h"
|
| +#include "content/public/browser/cookie_store_map.h"
|
| #include "content/public/browser/download_save_info.h"
|
| #include "content/public/browser/plugin_service_filter.h"
|
| #include "content/public/browser/resource_context.h"
|
| @@ -294,6 +295,7 @@ RenderMessageFilter::RenderMessageFilter(
|
| 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,
|
| @@ -302,6 +304,7 @@ RenderMessageFilter::RenderMessageFilter(
|
| plugin_service_(plugin_service),
|
| profile_data_directory_(browser_context->GetPath()),
|
| request_context_(request_context),
|
| + cookie_store_map_(cookie_store_map.Clone()),
|
| resource_context_(browser_context->GetResourceContext()),
|
| render_widget_helper_(render_widget_helper),
|
| incognito_(browser_context->IsOffTheRecord()),
|
| @@ -541,9 +544,10 @@ void RenderMessageFilter::OnSetCookie(const IPC::Message& message,
|
| url, first_party_for_cookies, cookie,
|
| resource_context_, render_process_id_, message.routing_id(),
|
| &options)) {
|
| - net::URLRequestContext* context = GetRequestContextForURL(url);
|
| // Pass a null callback since we don't care about when the 'set' completes.
|
| - context->cookie_store()->SetCookieWithOptionsAsync(
|
| + net::CookieStore* cookie_store =
|
| + cookie_store_map_->GetForScheme(url.scheme());
|
| + cookie_store->SetCookieWithOptionsAsync(
|
| url, cookie, options, net::CookieMonster::SetCookiesCallback());
|
| }
|
| }
|
| @@ -564,9 +568,8 @@ void RenderMessageFilter::OnGetCookies(const GURL& url,
|
| base::strlcpy(url_buf, url.spec().c_str(), arraysize(url_buf));
|
| base::debug::Alias(url_buf);
|
|
|
| - net::URLRequestContext* context = GetRequestContextForURL(url);
|
| net::CookieMonster* cookie_monster =
|
| - context->cookie_store()->GetCookieMonster();
|
| + cookie_store_map_->GetForScheme(url.scheme())->GetCookieMonster();
|
| cookie_monster->GetAllCookiesForURLAsync(
|
| url, base::Bind(&RenderMessageFilter::CheckPolicyForCookies, this, url,
|
| first_party_for_cookies, reply_msg));
|
| @@ -591,9 +594,8 @@ void RenderMessageFilter::OnGetRawCookies(
|
| // We check policy here to avoid sending back cookies that would not normally
|
| // be applied to outbound requests for the given URL. Since this cookie info
|
| // is visible in the developer tools, it is helpful to make it match reality.
|
| - net::URLRequestContext* context = GetRequestContextForURL(url);
|
| net::CookieMonster* cookie_monster =
|
| - context->cookie_store()->GetCookieMonster();
|
| + cookie_store_map_->GetForScheme(url.scheme())->GetCookieMonster();
|
| cookie_monster->GetAllCookiesForURLAsync(
|
| url, base::Bind(&RenderMessageFilter::SendGetRawCookiesResponse,
|
| this, reply_msg));
|
| @@ -606,8 +608,9 @@ void RenderMessageFilter::OnDeleteCookie(const GURL& url,
|
| if (!policy->CanAccessCookiesForOrigin(render_process_id_, url))
|
| return;
|
|
|
| - net::URLRequestContext* context = GetRequestContextForURL(url);
|
| - context->cookie_store()->DeleteCookieAsync(url, cookie_name, base::Closure());
|
| + net::CookieStore* cookie_store =
|
| + cookie_store_map_->GetForScheme(url.scheme());
|
| + cookie_store->DeleteCookieAsync(url, cookie_name, base::Closure());
|
| }
|
|
|
| void RenderMessageFilter::OnCookiesEnabled(
|
| @@ -867,19 +870,6 @@ void RenderMessageFilter::OnAllocateSharedMemory(
|
| buffer_size, PeerHandle(), handle);
|
| }
|
|
|
| -net::URLRequestContext* RenderMessageFilter::GetRequestContextForURL(
|
| - const GURL& url) {
|
| - DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
|
| -
|
| - net::URLRequestContext* context =
|
| - GetContentClient()->browser()->OverrideRequestContextForURL(
|
| - url, resource_context_);
|
| - if (!context)
|
| - context = request_context_->GetURLRequestContext();
|
| -
|
| - return context;
|
| -}
|
| -
|
| #if defined(OS_POSIX) && !defined(TOOLKIT_GTK) && !defined(OS_ANDROID)
|
| void RenderMessageFilter::OnAllocTransportDIB(
|
| uint32 size, bool cache_in_browser, TransportDIB::Handle* handle) {
|
| @@ -1038,14 +1028,15 @@ void RenderMessageFilter::CheckPolicyForCookies(
|
| const GURL& first_party_for_cookies,
|
| IPC::Message* reply_msg,
|
| const net::CookieList& cookie_list) {
|
| - net::URLRequestContext* context = GetRequestContextForURL(url);
|
| // Check the policy for get cookies, and pass cookie_list to the
|
| // TabSpecificContentSetting for logging purpose.
|
| if (GetContentClient()->browser()->AllowGetCookie(
|
| url, first_party_for_cookies, cookie_list, resource_context_,
|
| render_process_id_, reply_msg->routing_id())) {
|
| + net::CookieStore* cookie_store =
|
| + cookie_store_map_->GetForScheme(url.scheme());
|
| // Gets the cookies from cookie store if allowed.
|
| - context->cookie_store()->GetCookiesWithOptionsAsync(
|
| + cookie_store->GetCookiesWithOptionsAsync(
|
| url, net::CookieOptions(),
|
| base::Bind(&RenderMessageFilter::SendGetCookiesResponse,
|
| this, reply_msg));
|
|
|