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

Side by Side Diff: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

Issue 2136223003: ResourceDispatcherHostDelegate::OnResponseStarted doesn't need IPC::Sender (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix Created 4 years, 5 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 "chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate. h" 5 #include "chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate. h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 696 matching lines...) Expand 10 before | Expand all | Expand 10 after
707 request->GetExpectedContentSize(), info->GetChildID(), 707 request->GetExpectedContentSize(), info->GetChildID(),
708 info->GetRenderFrameID(), ix->second.extension_id, 708 info->GetRenderFrameID(), ix->second.extension_id,
709 ix->second.view_id, embedded)); 709 ix->second.view_id, embedded));
710 stream_target_info_.erase(request); 710 stream_target_info_.erase(request);
711 #endif 711 #endif
712 } 712 }
713 713
714 void ChromeResourceDispatcherHostDelegate::OnResponseStarted( 714 void ChromeResourceDispatcherHostDelegate::OnResponseStarted(
715 net::URLRequest* request, 715 net::URLRequest* request,
716 content::ResourceContext* resource_context, 716 content::ResourceContext* resource_context,
717 content::ResourceResponse* response, 717 content::ResourceResponse* response) {
718 IPC::Sender* sender) {
719 const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request); 718 const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request);
720 ProfileIOData* io_data = ProfileIOData::FromResourceContext(resource_context); 719 ProfileIOData* io_data = ProfileIOData::FromResourceContext(resource_context);
721 720
722 // See if the response contains the X-Chrome-Manage-Accounts header. If so 721 // See if the response contains the X-Chrome-Manage-Accounts header. If so
723 // show the profile avatar bubble so that user can complete signin/out action 722 // show the profile avatar bubble so that user can complete signin/out action
724 // the native UI. 723 // the native UI.
725 signin::ProcessMirrorResponseHeaderIfExists(request, io_data, 724 signin::ProcessMirrorResponseHeaderIfExists(request, io_data,
726 info->GetChildID(), 725 info->GetChildID(),
727 info->GetRouteID()); 726 info->GetRouteID());
728 727
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
849 data->SetDataReductionProxyData(data_reduction_proxy_data->DeepCopy()); 848 data->SetDataReductionProxyData(data_reduction_proxy_data->DeepCopy());
850 return data; 849 return data;
851 } 850 }
852 851
853 std::unique_ptr<net::ClientCertStore> 852 std::unique_ptr<net::ClientCertStore>
854 ChromeResourceDispatcherHostDelegate::CreateClientCertStore( 853 ChromeResourceDispatcherHostDelegate::CreateClientCertStore(
855 content::ResourceContext* resource_context) { 854 content::ResourceContext* resource_context) {
856 return ProfileIOData::FromResourceContext(resource_context)-> 855 return ProfileIOData::FromResourceContext(resource_context)->
857 CreateClientCertStore(); 856 CreateClientCertStore();
858 } 857 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698