Network Traffic Annotation added to DownloadURLParams.
Network traffic annotation argument added to DownloadURLParams. Annotation
argument was formerly passed separately along with each DownloadURLParam
causing complexities in plumbing.
Bug: 656607
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation
Change-Id: I707b1db05a9ae81f1965867b4a6af28c6ccd3e67
Reviewed-on: https://chromium-review.googlesource.com/530307
Reviewed-by: David Trainor <[email protected]>
Reviewed-by: Jochen Eisinger <[email protected]>
Commit-Queue: Ramin Halavati <[email protected]>
Cr-Commit-Position: refs/heads/master@{#478922}diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index ad735bf..40429d51 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -3213,7 +3213,8 @@
auto params = base::MakeUnique<DownloadUrlParameters>(
url, frame_host->GetProcess()->GetID(),
frame_host->GetRenderViewHost()->GetRoutingID(),
- frame_host->GetRoutingID(), storage_partition->GetURLRequestContext());
+ frame_host->GetRoutingID(), storage_partition->GetURLRequestContext(),
+ NO_TRAFFIC_ANNOTATION_YET);
params->set_referrer(referrer);
params->set_post_id(post_id);
if (post_id >= 0)
@@ -3233,7 +3234,7 @@
}
}
BrowserContext::GetDownloadManager(GetBrowserContext())
- ->DownloadUrl(std::move(params), NO_TRAFFIC_ANNOTATION_YET);
+ ->DownloadUrl(std::move(params));
}
void WebContentsImpl::GenerateMHTML(
