Hooked up NetLog to DownloadItem, DownloadFile, and FileStream.

content/browser/renderer_host/buffered_resource_handler.cc

Index: content/browser/renderer_host/buffered_resource_handler.cc
diff --git a/content/browser/renderer_host/buffered_resource_handler.cc b/content/browser/renderer_host/buffered_resource_handler.cc
index 5f59bb7c26e1a75d4ca54b5fe50c92afe41e524e..77517a687c5f78e30b52d90f6390accf71d59401 100644
--- a/content/browser/renderer_host/buffered_resource_handler.cc
+++ b/content/browser/renderer_host/buffered_resource_handler.cc
@@ -10,6 +10,7 @@
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/string_util.h"
+#include "content/browser/download/download_net_log_parameters.h"
 #include "content/browser/download/download_resource_handler.h"
 #include "content/browser/download/download_stats.h"
 #include "content/browser/download/download_types.h"
@@ -26,6 +27,7 @@
 #include "net/base/mime_sniffer.h"
 #include "net/base/mime_util.h"
 #include "net/base/net_errors.h"
+#include "net/base/net_log.h"
 #include "net/http/http_content_disposition.h"
 #include "net/http/http_response_headers.h"
 #include "webkit/plugins/webplugininfo.h"
@@ -303,12 +305,29 @@ bool BufferedResourceHandler::CompleteResponseStarted(int request_id) {
 
     info->set_is_download(true);
 
+    net::BoundNetLog bound_net_log =
+        net::BoundNetLog::Make(request_->net_log().net_log(),
+                               net::NetLog::SOURCE_DOWNLOAD);
+
+    bound_net_log.AddEvent(
+        net::NetLog::TYPE_DOWNLOAD_URL_REQUEST,
+        make_scoped_refptr(new net::NetLogSourceParameter(
+            "source_dependency",
+            request_->net_log().source())));

[mmenke, 2012/02/03 18:12:36]

It's a little weird to start with a singleton event, rather than a begin/end
event.

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

Yeah . . .

+
+    request_->net_log().AddEvent(
+        net::NetLog::TYPE_DOWNLOAD_STARTED,
+        make_scoped_refptr(new net::NetLogSourceParameter(
+            "source_dependency",
+            bound_net_log.source())));

[mmenke, 2012/02/03 18:12:36]

Could do all this in DownloadResourceHandler's constructor.  I'm not familiar
enough with the downloads code to know if that's a good idea or not,
architecturally, but you do duplicate this code in RDH.

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

Done.

+
     scoped_refptr<ResourceHandler> handler(
       new DownloadResourceHandler(host_,
                                   info->child_id(),
                                   info->route_id(),
                                   info->request_id(),
                                   request_->url(),
+                                  bound_net_log,
                                   host_->download_file_manager(),
                                   request_,
                                   DownloadResourceHandler::OnStartedCallback(),