Added net logging to DownloadItem.

chrome/browser/resources/net_internals/source_entry.js

 // 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.
 
 var SourceEntry = (function() {
   'use strict';
 
   /**
    * A SourceEntry gathers all log entries with the same source.
    *
@@ skipping 122 matching lines @@
           this.description_ = e.params.hostname;
           break;
         case LogSourceType.DOWNLOAD:
           switch (e.type) {
             case LogEventType.DOWNLOAD_FILE_RENAMED:
               this.description_ = e.params.new_filename;
               break;
             case LogEventType.DOWNLOAD_FILE_OPENED:
               this.description_ = e.params.file_name;
               break;
+            case LogEventType.DOWNLOAD_ITEM_ACTIVE:
+              this.description_ = e.params.intermediate_name;
+              break;
           }
           break;
         case LogSourceType.FILESTREAM:
           this.description_ = e.params.file_name;
           break;
       }
 
       if (this.description_ == undefined)
         this.description_ = '';
     },
@@ skipping 23 matching lines @@
       }
       if (this.entries_[0].source.type == LogSourceType.DOWNLOAD) {
         // If any rename occurred, use the last name
         e = this.findLastLogEntryByType_(LogEventType.DOWNLOAD_FILE_RENAMED);
         if (e != undefined)
           return e;
         // Otherwise, if the file was opened, use that name
         e = this.findLogEntryByType_(LogEventType.DOWNLOAD_FILE_OPENED);
         if (e != undefined)
           return e;
+        // History items are never opened, so use the activation info
+        e = this.findLogEntryByType_(LogEventType.DOWNLOAD_ITEM_ACTIVE);

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

Why aren't we using the DOWNLOAD_ITEM_ACTIVE in preference to the
DOWNLOAD_FILE_OPENED?  I think it's likely to happen earlier; is there some
reason to prefer file opening?

[ahendrickson, 2012/02/04 05:24:04]

This information is used to determine what to show for a description of the
event source.  The ACTIVE event doesn't have the right information for a new
download, but does for the one loaded from the history DB.

+        if (e != undefined)
+          return e;
       }
       if (this.entries_.length >= 2) {
         if (this.entries_[0].type == LogEventType.REQUEST_ALIVE ||
             this.entries_[0].type == LogEventType.SOCKET_POOL_CONNECT_JOB ||
             this.entries_[1].type == LogEventType.UDP_CONNECT) {
           return this.entries_[1];
         }
       }
       return this.entries_[0];
     },
@@ skipping 87 matching lines @@
      * Prints descriptive text about |entries_| to a new node added to the end
      * of |parent|.
      */
     printAsText: function(parent) {
       printLogEntriesAsText(this.entries_, parent);
     }
   };
 
   return SourceEntry;
 })();