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

Unified Diff: android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc

Issue 2334603002: Fix SSLUITest.TestBadHTTPSDownload with PlzNavigate. (Closed)
Patch Set: fix android temporarily 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 side-by-side diff with in-line comments
Download patch
Index: android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc
diff --git a/android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc b/android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc
index 460eb3664806a64633ad1719606a2062eb2d7fbc..47f5065081938f094c060b1d6a00d40c40c42888 100644
--- a/android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc
+++ b/android_webview/browser/renderer_host/aw_resource_dispatcher_host_delegate.cc
@@ -267,8 +267,6 @@ void AwResourceDispatcherHostDelegate::RequestComplete(
void AwResourceDispatcherHostDelegate::DownloadStarting(
net::URLRequest* request,
content::ResourceContext* resource_context,
- int child_id,
- int route_id,
bool is_content_initiated,
bool must_download,
ScopedVector<content::ResourceThrottle>* throttles) {
@@ -294,8 +292,10 @@ void AwResourceDispatcherHostDelegate::DownloadStarting(
const content::ResourceRequestInfo* request_info =
content::ResourceRequestInfo::ForRequest(request);
+ // TODO(jam): http://crbug.com/645983 we will need to make this map work with
+ // both RFH IDs and FTN IDs.
std::unique_ptr<AwContentsIoThreadClient> io_client =
- AwContentsIoThreadClient::FromID(child_id,
+ AwContentsIoThreadClient::FromID(request_info->GetChildID(),
request_info->GetRenderFrameID());
// POST request cannot be repeated in general, so prevent client from

Powered by Google App Engine
This is Rietveld 408576698