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

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

Issue 9113028: Substitute ResourceHandlers with dummy ResourceHandler while request is transferred (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed Darin's comments Created 8 years, 11 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
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 // 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 25 matching lines...) Expand all
36 #include "content/browser/renderer_host/buffered_resource_handler.h" 36 #include "content/browser/renderer_host/buffered_resource_handler.h"
37 #include "content/browser/renderer_host/cross_site_resource_handler.h" 37 #include "content/browser/renderer_host/cross_site_resource_handler.h"
38 #include "content/browser/renderer_host/redirect_to_file_resource_handler.h" 38 #include "content/browser/renderer_host/redirect_to_file_resource_handler.h"
39 #include "content/browser/renderer_host/render_view_host.h" 39 #include "content/browser/renderer_host/render_view_host.h"
40 #include "content/browser/renderer_host/render_view_host_delegate.h" 40 #include "content/browser/renderer_host/render_view_host_delegate.h"
41 #include "content/browser/renderer_host/resource_dispatcher_host_request_info.h" 41 #include "content/browser/renderer_host/resource_dispatcher_host_request_info.h"
42 #include "content/browser/renderer_host/resource_message_filter.h" 42 #include "content/browser/renderer_host/resource_message_filter.h"
43 #include "content/browser/renderer_host/resource_queue.h" 43 #include "content/browser/renderer_host/resource_queue.h"
44 #include "content/browser/renderer_host/resource_request_details.h" 44 #include "content/browser/renderer_host/resource_request_details.h"
45 #include "content/browser/renderer_host/sync_resource_handler.h" 45 #include "content/browser/renderer_host/sync_resource_handler.h"
46 #include "content/browser/renderer_host/doomed_resource_handler.h"
46 #include "content/browser/resource_context.h" 47 #include "content/browser/resource_context.h"
47 #include "content/browser/ssl/ssl_client_auth_handler.h" 48 #include "content/browser/ssl/ssl_client_auth_handler.h"
48 #include "content/browser/ssl/ssl_manager.h" 49 #include "content/browser/ssl/ssl_manager.h"
49 #include "content/browser/worker_host/worker_service_impl.h" 50 #include "content/browser/worker_host/worker_service_impl.h"
50 #include "content/common/resource_messages.h" 51 #include "content/common/resource_messages.h"
51 #include "content/common/view_messages.h" 52 #include "content/common/view_messages.h"
52 #include "content/public/browser/browser_thread.h" 53 #include "content/public/browser/browser_thread.h"
53 #include "content/public/browser/content_browser_client.h" 54 #include "content/public/browser/content_browser_client.h"
54 #include "content/public/browser/download_manager.h" 55 #include "content/public/browser/download_manager.h"
55 #include "content/public/browser/global_request_id.h" 56 #include "content/public/browser/global_request_id.h"
(...skipping 2183 matching lines...) Expand 10 before | Expand all | Expand 10 after
2239 bool ResourceDispatcherHost::allow_cross_origin_auth_prompt() { 2240 bool ResourceDispatcherHost::allow_cross_origin_auth_prompt() {
2240 return allow_cross_origin_auth_prompt_; 2241 return allow_cross_origin_auth_prompt_;
2241 } 2242 }
2242 2243
2243 void ResourceDispatcherHost::set_allow_cross_origin_auth_prompt(bool value) { 2244 void ResourceDispatcherHost::set_allow_cross_origin_auth_prompt(bool value) {
2244 allow_cross_origin_auth_prompt_ = value; 2245 allow_cross_origin_auth_prompt_ = value;
2245 } 2246 }
2246 2247
2247 void ResourceDispatcherHost::MarkAsTransferredNavigation( 2248 void ResourceDispatcherHost::MarkAsTransferredNavigation(
2248 const GlobalRequestID& transferred_request_id, 2249 const GlobalRequestID& transferred_request_id,
2249 net::URLRequest* ransferred_request) { 2250 net::URLRequest* transferred_request) {
2250 transferred_navigations_[transferred_request_id] = ransferred_request; 2251 transferred_navigations_[transferred_request_id] = transferred_request;
2252
2253 // If a URLRequest is transferred to a new RenderViewHost, its
2254 // ResourceHandlers should not receive any notifications because they may
2255 // depend on the state of the old RVH. We set a ResourceHandler that only
2256 // allows canceling requests, because on shutdown of the RDH all pending
2257 // requests are canceled. The RVH of requests that are being transferred may
2258 // be gone by that time. If the request is resumed, the ResoureHandlers are
2259 // substituted again.
2260 ResourceDispatcherHostRequestInfo* info = InfoForRequest(transferred_request);
2261 scoped_refptr<ResourceHandler> transferred_resource_handler(
2262 new DoomedResourceHandler(info->resource_handler()));
2263 info->set_resource_handler(transferred_resource_handler.get());
2251 } 2264 }
2252 2265
2253 bool ResourceDispatcherHost::IsTransferredNavigation( 2266 bool ResourceDispatcherHost::IsTransferredNavigation(
2254 const content::GlobalRequestID& transferred_request_id) const { 2267 const content::GlobalRequestID& transferred_request_id) const {
2255 return transferred_navigations_.find(transferred_request_id) != 2268 return transferred_navigations_.find(transferred_request_id) !=
2256 transferred_navigations_.end(); 2269 transferred_navigations_.end();
2257 } 2270 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698