Hooked up NetLog to DownloadItem, DownloadFile, and FileStream.

content/browser/download/download_manager_impl.cc

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 5b3bb3a3a02fcacc619d3296fcb09488da328063..9020c6b73dd16102076b8f991189b828a7a26658 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -127,19 +127,22 @@ namespace content {
 
 // static
 DownloadManager* DownloadManager::Create(
-    content::DownloadManagerDelegate* delegate) {
-  return new DownloadManagerImpl(delegate);
+    content::DownloadManagerDelegate* delegate,
+    net::NetLog* net_log) {
+  return new DownloadManagerImpl(delegate, net_log);
 }
 
 }  // namespace content
 
 DownloadManagerImpl::DownloadManagerImpl(
-    content::DownloadManagerDelegate* delegate)
+    content::DownloadManagerDelegate* delegate,
+    net::NetLog* net_log)
         : shutdown_needed_(false),
           browser_context_(NULL),
           file_manager_(NULL),
           delegate_(delegate),
-          largest_db_handle_in_history_(DownloadItem::kUninitializedHandle) {
+          largest_db_handle_in_history_(DownloadItem::kUninitializedHandle),
+          net_log_(net_log) {
 }
 
 DownloadManagerImpl::~DownloadManagerImpl() {
@@ -410,7 +413,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);
 
@@ -915,7 +918,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;
@@ -1173,3 +1176,7 @@ void DownloadManagerImpl::DownloadOpened(DownloadItem* download) {
 void DownloadManagerImpl::SetFileManager(DownloadFileManager* file_manager) {
   file_manager_ = file_manager;
 }
+
+net::NetLog* DownloadManagerImpl::GetNetLog() {
+  return net_log_;

[Randy Smith (Not in Mondays), 2012/02/07 15:20:39]

I'd argue that there's no point to this routine; it's just cluttering the DMI
interface (admittedly at the private level, so it isn't as painful, but it's
still clutter).  DMI methods are allowed to know about DMI member variables, and
net_log_ is just as understandable as GetNetLog().  More, even, because the
audience knows that there's no magic going on in GetNetLog().

[ahendrickson, 2012/02/07 17:58:38]

Removed.

+}