Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(219)

Unified Diff: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

Issue 481593002: Remove child/route parameters from ShouldBeginRequest and RequestBeginning. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: android build Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
diff --git a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
index 641ea182807c25be3cd536f22800967306d79ba5..35144614b1b21320dc62173b2c04a42764e6f6f7 100644
--- a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
+++ b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
@@ -271,8 +271,6 @@ ChromeResourceDispatcherHostDelegate::~ChromeResourceDispatcherHostDelegate() {
}
bool ChromeResourceDispatcherHostDelegate::ShouldBeginRequest(
- int child_id,
- int route_id,
const std::string& method,
const GURL& url,
ResourceType resource_type,
@@ -299,8 +297,6 @@ void ChromeResourceDispatcherHostDelegate::RequestBeginning(
content::ResourceContext* resource_context,
content::AppCacheService* appcache_service,
ResourceType resource_type,
- int child_id,
- int route_id,
ScopedVector<content::ResourceThrottle>* throttles) {
const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request);
bool is_prerendering =
@@ -395,8 +391,7 @@ void ChromeResourceDispatcherHostDelegate::RequestBeginning(
#endif
signin::AppendMirrorRequestHeaderIfPossible(
- request, GURL() /* redirect_url */,
- io_data, info->GetChildID(), info->GetRouteID());
+ request, GURL() /* redirect_url */, io_data);
AppendStandardResourceThrottles(request,
resource_context,
@@ -692,9 +687,10 @@ void ChromeResourceDispatcherHostDelegate::OnRequestRedirected(
content::ResourceContext* resource_context,
content::ResourceResponse* response) {
ProfileIOData* io_data = ProfileIOData::FromResourceContext(resource_context);
- const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request);
#if defined(ENABLE_ONE_CLICK_SIGNIN)
+ const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request);
+
// See if the response contains the Google-Accounts-SignIn header. If so,
// then the user has just finished signing in, and the server is allowing the
// browser to suggest connecting the user's profile to the account.
@@ -709,8 +705,7 @@ void ChromeResourceDispatcherHostDelegate::OnRequestRedirected(
// response and let Chrome handle the action with native UI. The only
// exception is requests from gaia webview, since the native profile
// management UI is built on top of it.
- signin::AppendMirrorRequestHeaderIfPossible(request, redirect_url, io_data,
- info->GetChildID(), info->GetRouteID());
+ signin::AppendMirrorRequestHeaderIfPossible(request, redirect_url, io_data);
#if defined(ENABLE_CONFIGURATION_POLICY)
if (io_data->policy_header_helper())

Powered by Google App Engine
This is Rietveld 408576698