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

Unified Diff: content/browser/download/download_manager_impl.cc

Issue 2351513002: net: rename BoundNetLog to NetLogWithSource (Closed)
Patch Set: REBASE 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: content/browser/download/download_manager_impl.cc
diff --git a/content/browser/download/download_manager_impl.cc b/content/browser/download/download_manager_impl.cc
index b4fc75689a0c5afec326b36d94f36168c3ce1b2a..e4bf7b79a813d1fcdd6e71df49cd90895d05696b 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -90,7 +90,7 @@ std::unique_ptr<UrlDownloader, BrowserThread::DeleteOnIOThread> BeginDownload(
// Otherwise, create an interrupted download.
std::unique_ptr<DownloadCreateInfo> failed_created_info(
- new DownloadCreateInfo(base::Time::Now(), net::BoundNetLog(),
+ new DownloadCreateInfo(base::Time::Now(), net::NetLogWithSource(),
base::WrapUnique(new DownloadSaveInfo)));
failed_created_info->url_chain.push_back(params->url());
failed_created_info->result = reason;
@@ -138,7 +138,7 @@ class DownloadItemFactoryImpl : public DownloadItemFactory {
DownloadDangerType danger_type,
DownloadInterruptReason interrupt_reason,
bool opened,
- const net::BoundNetLog& bound_net_log) override {
+ const net::NetLogWithSource& bound_net_log) override {
return new DownloadItemImpl(
delegate, guid, download_id, current_path, target_path, url_chain,
referrer_url, site_url, tab_url, tab_refererr_url, mime_type,
@@ -151,7 +151,7 @@ class DownloadItemFactoryImpl : public DownloadItemFactory {
DownloadItemImplDelegate* delegate,
uint32_t download_id,
const DownloadCreateInfo& info,
- const net::BoundNetLog& bound_net_log) override {
+ const net::NetLogWithSource& bound_net_log) override {
return new DownloadItemImpl(delegate, download_id, info, bound_net_log);
}
@@ -162,7 +162,7 @@ class DownloadItemFactoryImpl : public DownloadItemFactory {
const GURL& url,
const std::string& mime_type,
std::unique_ptr<DownloadRequestHandleInterface> request_handle,
- const net::BoundNetLog& bound_net_log) override {
+ const net::NetLogWithSource& bound_net_log) override {
return new DownloadItemImpl(delegate, download_id, path, url, mime_type,
std::move(request_handle), bound_net_log);
}
@@ -192,8 +192,8 @@ DownloadItemImpl* DownloadManagerImpl::CreateActiveItem(
const DownloadCreateInfo& info) {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
DCHECK(!base::ContainsKey(downloads_, id));
- net::BoundNetLog bound_net_log =
- net::BoundNetLog::Make(net_log_, net::NetLogSourceType::DOWNLOAD);
+ net::NetLogWithSource bound_net_log =
+ net::NetLogWithSource::Make(net_log_, net::NetLogSourceType::DOWNLOAD);
DownloadItemImpl* download =
item_factory_->CreateActiveItem(this, id, info, bound_net_log);
downloads_[id] = download;
@@ -362,12 +362,10 @@ void DownloadManagerImpl::StartDownloadWithId(
if (info->result == DOWNLOAD_INTERRUPT_REASON_NONE) {
DCHECK(stream.get());
- download_file.reset(
- file_factory_->CreateFile(std::move(info->save_info),
- default_download_directory,
- std::move(stream),
- download->GetBoundNetLog(),
- download->DestinationObserverAsWeakPtr()));
+ download_file.reset(file_factory_->CreateFile(
+ std::move(info->save_info), default_download_directory,
+ std::move(stream), download->GetNetLogWithSource(),
+ download->DestinationObserverAsWeakPtr()));
}
// It is important to leave info->save_info intact in the case of an interrupt
// so that the DownloadItem can salvage what it can out of a failed resumption
@@ -444,8 +442,8 @@ void DownloadManagerImpl::CreateSavePackageDownloadItemWithId(
DCHECK_CURRENTLY_ON(BrowserThread::UI);
DCHECK_NE(content::DownloadItem::kInvalidId, id);
DCHECK(!base::ContainsKey(downloads_, id));
- net::BoundNetLog bound_net_log =
- net::BoundNetLog::Make(net_log_, net::NetLogSourceType::DOWNLOAD);
+ net::NetLogWithSource bound_net_log =
+ net::NetLogWithSource::Make(net_log_, net::NetLogSourceType::DOWNLOAD);
DownloadItemImpl* download_item = item_factory_->CreateSavePageItem(
this, id, main_file_path, page_url, mime_type, std::move(request_handle),
bound_net_log);
@@ -694,7 +692,7 @@ DownloadItem* DownloadManagerImpl::CreateDownloadItem(
site_url, tab_url, tab_refererr_url, mime_type, original_mime_type,
start_time, end_time, etag, last_modified, received_bytes, total_bytes,
hash, state, danger_type, interrupt_reason, opened,
- net::BoundNetLog::Make(net_log_, net::NetLogSourceType::DOWNLOAD));
+ net::NetLogWithSource::Make(net_log_, net::NetLogSourceType::DOWNLOAD));
downloads_[id] = item;
downloads_by_guid_[guid] = item;
FOR_EACH_OBSERVER(Observer, observers_, OnDownloadCreated(this, item));

Powered by Google App Engine
This is Rietveld 408576698