Hooked up NetLog to DownloadItem, DownloadFile, and FileStream.

content/browser/download/download_item_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/download/download_item_impl.h"
 
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 220 matching lines @@
       open_when_complete_(false),
       file_externally_removed_(false),
       safety_state_(SAFE),
       auto_opened_(false),
       is_otr_(is_otr),
       is_temporary_(!info.save_info.file_path.empty()),
       all_data_saved_(false),
       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);
 }
 
 // Constructing for the "Save Page As..." feature:
 DownloadItemImpl::DownloadItemImpl(Delegate* delegate,
                                    const FilePath& path,
                                    const GURL& url,
                                    bool is_otr,
@@ skipping 850 matching lines @@
   std::map<const void*, ExternalData*>::iterator it =
       external_data_map_.find(key);
 
   if (it == external_data_map_.end()) {
     external_data_map_[key] = data;
   } else if (it->second != data) {
     delete it->second;
     it->second = data;
   }
 }