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

Side by Side Diff: content/browser/loader/resource_dispatcher_host_impl.h

Issue 2239273002: Don't use SSLStatus from FrameHostMsg_DidCommitProvisionalLoad and instead cache it on the browser … (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments Created 4 years, 3 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 // This is the browser side of the resource dispatcher, it receives requests 5 // This is the browser side of the resource dispatcher, it receives requests
6 // from the child process (i.e. [Renderer, Plugin, Worker]ProcessHost), and 6 // from the child process (i.e. [Renderer, Plugin, Worker]ProcessHost), and
7 // dispatches them to URLRequests. It then forwards the messages from the 7 // dispatches them to URLRequests. It then forwards the messages from the
8 // URLRequests back to the correct process for handling. 8 // URLRequests back to the correct process for handling.
9 // 9 //
10 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading 10 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading
(...skipping 581 matching lines...) Expand 10 before | Expand all | Expand 10 after
592 592
593 int BuildLoadFlagsForRequest(const ResourceRequest& request_data, 593 int BuildLoadFlagsForRequest(const ResourceRequest& request_data,
594 int child_id, 594 int child_id,
595 bool is_sync_load); 595 bool is_sync_load);
596 596
597 // The certificate on a ResourceResponse is associated with a 597 // The certificate on a ResourceResponse is associated with a
598 // particular renderer process. As a transfer to a new process 598 // particular renderer process. As a transfer to a new process
599 // completes, the stored certificate has to be updated to reflect the 599 // completes, the stored certificate has to be updated to reflect the
600 // new renderer process. 600 // new renderer process.
601 void UpdateResponseCertificateForTransfer(ResourceResponse* response, 601 void UpdateResponseCertificateForTransfer(ResourceResponse* response,
602 const net::SSLInfo& ssl_info, 602 net::URLRequest* request,
603 int child_id); 603 ResourceRequestInfoImpl* info);
604 604
605 CertStore* GetCertStore(); 605 CertStore* GetCertStore();
606 606
607 // Consults the RendererSecurity policy to determine whether the 607 // Consults the RendererSecurity policy to determine whether the
608 // ResourceDispatcherHostImpl should service this request. A request might 608 // ResourceDispatcherHostImpl should service this request. A request might
609 // be disallowed if the renderer is not authorized to retrieve the request 609 // be disallowed if the renderer is not authorized to retrieve the request
610 // URL or if the renderer is attempting to upload an unauthorized file. 610 // URL or if the renderer is attempting to upload an unauthorized file.
611 bool ShouldServiceRequest(int process_type, 611 bool ShouldServiceRequest(int process_type,
612 int child_id, 612 int child_id,
613 const ResourceRequest& request_data, 613 const ResourceRequest& request_data,
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
708 708
709 // Used to invoke an interceptor for the HTTP header. 709 // Used to invoke an interceptor for the HTTP header.
710 HeaderInterceptorMap http_header_interceptor_map_; 710 HeaderInterceptorMap http_header_interceptor_map_;
711 711
712 DISALLOW_COPY_AND_ASSIGN(ResourceDispatcherHostImpl); 712 DISALLOW_COPY_AND_ASSIGN(ResourceDispatcherHostImpl);
713 }; 713 };
714 714
715 } // namespace content 715 } // namespace content
716 716
717 #endif // CONTENT_BROWSER_LOADER_RESOURCE_DISPATCHER_HOST_IMPL_H_ 717 #endif // CONTENT_BROWSER_LOADER_RESOURCE_DISPATCHER_HOST_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698