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

Side by Side Diff: content/browser/renderer_host/async_resource_handler.cc

Issue 10827103: Speculative resource prefetching - fixing missing navigations due to server side redirects. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Passing redirect_url as an argument to ResourceDispatcherHostDelegate::OnRequestRedirected. Created 8 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/async_resource_handler.h" 5 #include "content/browser/renderer_host/async_resource_handler.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/debug/alias.h" 10 #include "base/debug/alias.h"
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 position, size)); 137 position, size));
138 } 138 }
139 139
140 bool AsyncResourceHandler::OnRequestRedirected(int request_id, 140 bool AsyncResourceHandler::OnRequestRedirected(int request_id,
141 const GURL& new_url, 141 const GURL& new_url,
142 ResourceResponse* response, 142 ResourceResponse* response,
143 bool* defer) { 143 bool* defer) {
144 *defer = did_defer_ = true; 144 *defer = did_defer_ = true;
145 145
146 if (rdh_->delegate()) { 146 if (rdh_->delegate()) {
147 rdh_->delegate()->OnRequestRedirected(request_, filter_->resource_context(), 147 rdh_->delegate()->OnRequestRedirected(new_url,
James Hawkins 2012/08/02 18:17:50 Optional nit: Condense parameters to save lines.
Shishir 2012/08/02 18:25:43 Done.
148 request_,
149 filter_->resource_context(),
148 response); 150 response);
149 } 151 }
150 152
151 DevToolsNetLogObserver::PopulateResponseInfo(request_, response); 153 DevToolsNetLogObserver::PopulateResponseInfo(request_, response);
152 response->head.request_start = request_->creation_time(); 154 response->head.request_start = request_->creation_time();
153 response->head.response_start = TimeTicks::Now(); 155 response->head.response_start = TimeTicks::Now();
154 return filter_->Send(new ResourceMsg_ReceivedRedirect( 156 return filter_->Send(new ResourceMsg_ReceivedRedirect(
155 routing_id_, request_id, new_url, response->head)); 157 routing_id_, request_id, new_url, response->head));
156 } 158 }
157 159
(...skipping 193 matching lines...) Expand 10 before | Expand all | Expand 10 after
351 } 353 }
352 354
353 void AsyncResourceHandler::ResumeIfDeferred() { 355 void AsyncResourceHandler::ResumeIfDeferred() {
354 if (did_defer_) { 356 if (did_defer_) {
355 did_defer_ = false; 357 did_defer_ = false;
356 controller()->Resume(); 358 controller()->Resume();
357 } 359 }
358 } 360 }
359 361
360 } // namespace content 362 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698