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

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

Issue 6901128: Cancel prerenders that spawn post requests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 9 years, 7 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 | Annotate | Revision Log
« no previous file with comments | « chrome/test/data/prerender/prerender_xhr_trace.html ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading 5 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading
6 6
7 #include "content/browser/renderer_host/resource_dispatcher_host.h" 7 #include "content/browser/renderer_host/resource_dispatcher_host.h"
8 8
9 #include <set> 9 #include <set>
10 #include <vector> 10 #include <vector>
(...skipping 393 matching lines...) Expand 10 before | Expand all | Expand 10 after
404 AbortRequestBeforeItStarts(filter_, sync_result, route_id, request_id); 404 AbortRequestBeforeItStarts(filter_, sync_result, route_id, request_id);
405 return; 405 return;
406 } 406 }
407 if (!ResourceDispatcherHost::is_prefetch_enabled()) { 407 if (!ResourceDispatcherHost::is_prefetch_enabled()) {
408 AbortRequestBeforeItStarts(filter_, sync_result, route_id, request_id); 408 AbortRequestBeforeItStarts(filter_, sync_result, route_id, request_id);
409 return; 409 return;
410 } 410 }
411 if (prerender::PrerenderManager::IsPrerenderingPossible()) { 411 if (prerender::PrerenderManager::IsPrerenderingPossible()) {
412 BrowserThread::PostTask( 412 BrowserThread::PostTask(
413 BrowserThread::UI, FROM_HERE, 413 BrowserThread::UI, FROM_HERE,
414 NewRunnableFunction(prerender::HandlePrefetchTagOnUIThread, 414 NewRunnableFunction(prerender::HandlePrefetchTag,
415 resource_context.prerender_manager(), 415 resource_context.prerender_manager(),
416 std::make_pair(child_id, route_id), 416 child_id,
417 route_id,
417 request_data.url, 418 request_data.url,
418 referrer, 419 referrer,
419 is_prerendering)); 420 is_prerendering));
420 AbortRequestBeforeItStarts(filter_, sync_result, route_id, request_id); 421 AbortRequestBeforeItStarts(filter_, sync_result, route_id, request_id);
421 return; 422 return;
422 } 423 }
423 // Otherwise, treat like a normal request, and fall-through. 424 // Otherwise, treat like a normal request, and fall-through.
424 } 425 }
425 426
427 // Abort any prerenders that spawn requests that use invalid HTTP methods.
428 if (is_prerendering &&
429 !prerender::PrerenderManager::IsValidHttpMethod(request_data.method)) {
430 BrowserThread::PostTask(
431 BrowserThread::UI, FROM_HERE,
432 NewRunnableFunction(
433 prerender::DestroyPreloadForRenderView,
434 resource_context.prerender_manager(),
435 child_id,
436 route_id,
437 prerender::FINAL_STATUS_INVALID_HTTP_METHOD));
438 AbortRequestBeforeItStarts(filter_, sync_result, route_id, request_id);
439 return;
440 }
426 441
427 // Construct the event handler. 442 // Construct the event handler.
428 scoped_refptr<ResourceHandler> handler; 443 scoped_refptr<ResourceHandler> handler;
429 if (sync_result) { 444 if (sync_result) {
430 handler = new SyncResourceHandler( 445 handler = new SyncResourceHandler(
431 filter_, request_data.url, sync_result, this); 446 filter_, request_data.url, sync_result, this);
432 } else { 447 } else {
433 handler = new AsyncResourceHandler( 448 handler = new AsyncResourceHandler(
434 filter_, route_id, request_data.url, 449 filter_, route_id, request_data.url,
435 resource_context.host_zoom_map(), 450 resource_context.host_zoom_map(),
(...skipping 1553 matching lines...) Expand 10 before | Expand all | Expand 10 after
1989 return is_prefetch_enabled_; 2004 return is_prefetch_enabled_;
1990 } 2005 }
1991 2006
1992 // static 2007 // static
1993 void ResourceDispatcherHost::set_is_prefetch_enabled(bool value) { 2008 void ResourceDispatcherHost::set_is_prefetch_enabled(bool value) {
1994 is_prefetch_enabled_ = value; 2009 is_prefetch_enabled_ = value;
1995 } 2010 }
1996 2011
1997 // static 2012 // static
1998 bool ResourceDispatcherHost::is_prefetch_enabled_ = false; 2013 bool ResourceDispatcherHost::is_prefetch_enabled_ = false;
OLDNEW
« no previous file with comments | « chrome/test/data/prerender/prerender_xhr_trace.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698