Added net logging to DownloadItem.

chrome/browser/net/passive_log_collector.cc

Index: chrome/browser/net/passive_log_collector.cc
diff --git a/chrome/browser/net/passive_log_collector.cc b/chrome/browser/net/passive_log_collector.cc
index 86af06dbf368f1e59a25879976e882b8dae1cbe9..15ec6c0318a3a00a840adc6210f479820116d0a6 100644
--- a/chrome/browser/net/passive_log_collector.cc
+++ b/chrome/browser/net/passive_log_collector.cc
@@ -869,7 +869,7 @@ PassiveLogCollector::DowloadTracker::DoAddEntry(
     const ChromeNetLog::Entry& entry,
     SourceInfo* out_info) {
   AddEntryToSourceInfo(entry, out_info);
-  if (entry.type == net::NetLog::TYPE_DOWNLOAD_FILE_OPENED &&
+  if (entry.type == net::NetLog::TYPE_DOWNLOAD_ITEM_ACTIVE &&

[Randy Smith (Not in Mondays), 2012/02/03 18:23:37]

Sorry, my lack of knowledge of netlog, but what is the purpose of this routine? 
If it's to close off dangling begin events, aren't there other begin events we
need to close off?

[mmenke, 2012/02/03 18:27:54]

Two purposes:  To avoid keeping spammy events around and to know when we've seen
a source's last close event, so we know when we can clean it up.

       entry.phase == net::NetLog::PHASE_END) {
     return ACTION_MOVE_TO_GRAVEYARD;
   }