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

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

Issue 9296012: Hooked up NetLog to DownloadItem, DownloadFile, and FileStream. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merged with parent Created 8 years, 11 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 edcfb5f3f587aac986e429a13324c3c734a46400..194c70f82b3166abc979bdd98820f5e9f75d10b9 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -116,15 +116,17 @@ namespace content {
// static
DownloadManager* DownloadManager::Create(
content::DownloadManagerDelegate* delegate,
- DownloadStatusUpdater* status_updater) {
- return new DownloadManagerImpl(delegate, status_updater);
+ DownloadStatusUpdater* status_updater,
+ net::NetLog* net_log) {
+ return new DownloadManagerImpl(delegate, status_updater, net_log);
}
} // namespace content
DownloadManagerImpl::DownloadManagerImpl(
content::DownloadManagerDelegate* delegate,
- DownloadStatusUpdater* status_updater)
+ DownloadStatusUpdater* status_updater,
+ net::NetLog* net_log)
: shutdown_needed_(false),
browser_context_(NULL),
file_manager_(NULL),
@@ -132,7 +134,8 @@ DownloadManagerImpl::DownloadManagerImpl(
? status_updater->AsWeakPtr()
: base::WeakPtr<DownloadStatusUpdater>()),
delegate_(delegate),
- largest_db_handle_in_history_(DownloadItem::kUninitializedHandle) {
+ largest_db_handle_in_history_(DownloadItem::kUninitializedHandle),
+ net_log_(net_log) {
// NOTE(benjhayden): status_updater may be NULL when using
// TestingBrowserProcess.
if (status_updater_.get() != NULL)
@@ -411,7 +414,7 @@ DownloadItem* DownloadManagerImpl::CreateSavePackageDownloadItem(
bool is_otr,
DownloadItem::Observer* observer) {
DownloadItem* download = new DownloadItemImpl(
- this, main_file_path, page_url, is_otr, GetNextId(), NULL);
+ this, main_file_path, page_url, is_otr, GetNextId(), GetNetLog());
download->AddObserver(observer);
@@ -949,7 +952,7 @@ void DownloadManagerImpl::OnPersistentStoreQueryComplete(
for (size_t i = 0; i < entries->size(); ++i) {
DownloadItem* download = new DownloadItemImpl(
- this, GetNextId(), entries->at(i), NULL);
+ this, GetNextId(), entries->at(i), GetNetLog());
CHECK_96627(!ContainsKey(history_downloads_, download->GetDbHandle()));
downloads_.insert(download);
history_downloads_[download->GetDbHandle()] = download;
@@ -1207,3 +1210,7 @@ void DownloadManagerImpl::DownloadOpened(DownloadItem* download) {
void DownloadManagerImpl::SetFileManager(DownloadFileManager* file_manager) {
file_manager_ = file_manager;
}
+
+net::NetLog* DownloadManagerImpl::GetNetLog() {
+ return net_log_;
+}

Powered by Google App Engine
This is Rietveld 408576698