Hooked up NetLog to DownloadItem, DownloadFile, and FileStream.

content/browser/download/download_manager_impl.h

Index: content/browser/download/download_manager_impl.h
diff --git a/content/browser/download/download_manager_impl.h b/content/browser/download/download_manager_impl.h
index 420bdbacd9ef3fb046490c84e17d2b3c794f065a..ab91d59620aff3c8d599bb8fe68a962c6f5b132c 100644
--- a/content/browser/download/download_manager_impl.h
+++ b/content/browser/download/download_manager_impl.h
@@ -30,7 +30,8 @@ class CONTENT_EXPORT DownloadManagerImpl
       public DownloadStatusUpdaterDelegate {
  public:
   DownloadManagerImpl(content::DownloadManagerDelegate* delegate,
-                      DownloadStatusUpdater* status_updater);
+                      DownloadStatusUpdater* status_updater,
+                      net::NetLog* net_log);
 
   // content::DownloadManager functions.
   virtual void Shutdown() OVERRIDE;
@@ -96,6 +97,7 @@ class CONTENT_EXPORT DownloadManagerImpl
   virtual content::DownloadManagerDelegate* delegate() const OVERRIDE;
   virtual void SetDownloadManagerDelegate(
       content::DownloadManagerDelegate* delegate) OVERRIDE;
+  virtual net::NetLog* GetNetLog() OVERRIDE;

[Randy Smith (Not in Mondays), 2012/02/03 19:22:13]

Why is this public?  I may have missed it, but I didn't see any place it was
used outside download_manager_impl.h, and it doesn't feel like it's part of the
DownloadManager interface.

[ahendrickson, 2012/02/05 05:06:53]

It is no longer part of the DownloadManager interface.

 
   // Overridden from DownloadItemImpl::Delegate
   // (Note that |GetBrowserContext| are present in both interfaces.)
@@ -263,6 +265,8 @@ class CONTENT_EXPORT DownloadManagerImpl
   // For debugging only.
   int64 largest_db_handle_in_history_;
 
+  net::NetLog* net_log_;
+
   DISALLOW_COPY_AND_ASSIGN(DownloadManagerImpl);
 };