Hooked up NetLog to DownloadItem, DownloadFile, and FileStream.

content/public/browser/download_manager.h

 // 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.
 //
 // The DownloadManager object manages the process of downloading, including
 // updates to the history system and providing the information for displaying
 // the downloads view in the Destinations tab. There is one DownloadManager per
 // active browser context in Chrome.
 //
 // Download observers:
@@ skipping 22 matching lines @@
 
 #include "base/basictypes.h"
 #include "base/file_path.h"
 #include "base/gtest_prod_util.h"
 #include "base/message_loop_helpers.h"
 #include "base/time.h"
 #include "content/browser/download/interrupt_reasons.h"
 #include "content/public/browser/download_id.h"
 #include "content/public/browser/download_item.h"
 #include "content/public/browser/browser_thread.h"
+#include "net/base/net_log.h"
 #include "net/base/net_errors.h"
 
 class DownloadFileManager;
 class DownloadManagerTest;
 class DownloadRequestHandle;
 class GURL;
 class TabContents;
 struct DownloadCreateInfo;
 struct DownloadRetrieveInfo;
 struct DownloadSaveInfo;
 
+namespace net {
+class NetLog;
+}  // namespace net

[Randy Smith (Not in Mondays), 2012/02/07 23:21:31]

You don't need the terminal comment if it's this short (at least based on what I
see in other files).

[ahendrickson, 2012/02/07 23:58:54]

Removed.

+
 namespace content {
 class BrowserContext;
 class DownloadManagerDelegate;
 class DownloadQuery;
 class WebContents;
 
 // Browser's download manager: manages all downloads and destination view.
 class CONTENT_EXPORT DownloadManager
     : public base::RefCountedThreadSafe<DownloadManager> {
  public:
   virtual ~DownloadManager() {}
 
   static DownloadManager* Create(
-      DownloadManagerDelegate* delegate);
+      DownloadManagerDelegate* delegate,
+      net::NetLog* net_log);
 
   // Shutdown the download manager. Must be called before destruction.
   virtual void Shutdown() = 0;
 
   // Interface to implement for observers that wish to be informed of changes
   // to the DownloadManager's collection of downloads.
   class CONTENT_EXPORT Observer {
    public:
     // New or deleted download, observers should query us for the current set
     // of downloads.
@@ skipping 121 matching lines @@
                                             int64 db_handle) = 0;
 
   // The number of in progress (including paused) downloads.
   virtual int InProgressCount() const = 0;
 
   virtual content::BrowserContext* GetBrowserContext() const = 0;
 
   virtual FilePath LastDownloadPath() = 0;
 
   // Creates the download item.  Must be called on the UI thread.
-  virtual void CreateDownloadItem(
+  virtual net::BoundNetLog CreateDownloadItem(

[Randy Smith (Not in Mondays), 2012/02/07 23:19:14]

Could you document the return value?

[ahendrickson, 2012/02/07 23:58:54]

Done.

       DownloadCreateInfo* info,
       const DownloadRequestHandle& request_handle) = 0;
 
   // Creates a download item for the SavePackage system.
   // Must be called on the UI thread.  Note that the DownloadManager
   // retains ownership.
   virtual DownloadItem* CreateSavePackageDownloadItem(
       const FilePath& main_file_path,
       const GURL& page_url,
       bool is_otr,
@@ skipping 57 matching lines @@
   friend class base::RefCountedThreadSafe<
       DownloadManager, content::BrowserThread::DeleteOnUIThread>;
   friend struct content::BrowserThread::DeleteOnThread<
       content::BrowserThread::UI>;
   friend class base::DeleteHelper<DownloadManager>;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_DOWNLOAD_MANAGER_H_