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

Unified Diff: components/download/content/download_driver_impl.cc

Issue 2860593003: Refactoring DownloadManager::DownloadURL to add proper annotation. (Closed)
Patch Set: Comment addressed. Created 3 years, 7 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
« no previous file with comments | « components/download/content/download_driver_impl.h ('k') | components/download/internal/download_driver.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/download/content/download_driver_impl.cc
diff --git a/components/download/content/download_driver_impl.cc b/components/download/content/download_driver_impl.cc
index 0d1d2811430ef5cc79a42201106575d270f3b872..17f684008564bfa5345120187580aeef7b603e20 100644
--- a/components/download/content/download_driver_impl.cc
+++ b/components/download/content/download_driver_impl.cc
@@ -81,7 +81,9 @@ bool DownloadDriverImpl::IsReady() const {
return client_ && download_manager_;
}
-void DownloadDriverImpl::Start(const DownloadParams& params) {
+void DownloadDriverImpl::Start(
+ const DownloadParams& params,
+ const net::NetworkTrafficAnnotationTag& traffic_annotation) {
DCHECK(!params.request_params.url.is_empty());
DCHECK(!params.guid.empty());
if (!download_manager_)
@@ -106,7 +108,8 @@ void DownloadDriverImpl::Start(const DownloadParams& params) {
download_url_params->set_method(params.request_params.method);
download_url_params->set_file_path(file_dir_.AppendASCII(params.guid));
- download_manager_->DownloadUrl(std::move(download_url_params));
+ download_manager_->DownloadUrl(std::move(download_url_params),
+ traffic_annotation);
}
void DownloadDriverImpl::Cancel(const std::string& guid) {
« no previous file with comments | « components/download/content/download_driver_impl.h ('k') | components/download/internal/download_driver.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698