Hooked up NetLog to DownloadItem, DownloadFile, and FileStream.

content/browser/download/download_item_impl.cc

Index: content/browser/download/download_item_impl.cc
diff --git a/content/browser/download/download_item_impl.cc b/content/browser/download/download_item_impl.cc
index fd8a622a9279ef07664a089bf7ab46b47ae1fc01..f8b2141acb31a16c1ad7596e18192f11800fba13 100644
--- a/content/browser/download/download_item_impl.cc
+++ b/content/browser/download/download_item_impl.cc
@@ -238,7 +238,7 @@ DownloadItemImpl::DownloadItemImpl(
       opened_(false),
       open_enabled_(true),
       delegate_delayed_complete_(false),
-      bound_net_log_(net::BoundNetLog()) {
+      bound_net_log_(info.bound_net_log) {

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

Ah, now I understand the difference in constructors.  I still think that making
them parallel (all taking a BoundNetLog()) makes sense, and I'd like a comment
in the .h file indicating that the BoundNetLog for the regular download
constructor comes from the info structure.

I wish we could combine these constructors :-{.

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

They now take BoundNetLog references instead of NetLog pointers, and
DownloadManagerImpl creates the BoundNetLogs.

   delegate_->Attach();
   Init(true /* actively downloading */,
        download_net_logs::SRC_NEW_DOWNLOAD);