Include high-fidelity metadata about a download in incident reports.

chrome/browser/download/download_service.cc

Index: chrome/browser/download/download_service.cc
diff --git a/chrome/browser/download/download_service.cc b/chrome/browser/download/download_service.cc
index f69dcc4eb262a02b7a86d652bdce42847fe479a6..3d42abda1719e4be7c6fc3e898c3851f0d2c575d 100644
--- a/chrome/browser/download/download_service.cc
+++ b/chrome/browser/download/download_service.cc
@@ -16,6 +16,7 @@
 #include "chrome/browser/net/chrome_net_log.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/safe_browsing/safe_browsing_service.h"
 #include "content/public/browser/download_manager.h"
 
 #if defined(ENABLE_EXTENSIONS)
@@ -64,6 +65,9 @@ ChromeDownloadManagerDelegate* DownloadService::GetDownloadManagerDelegate() {
         manager,
         scoped_ptr<DownloadHistory::HistoryAdapter>(
             new DownloadHistory::HistoryAdapter(history))));
+
+    // Include this download manager in the set monitored by safe browsing.
+    g_browser_process->safe_browsing_service()->AddDownloadManager(manager);

[mattm, 2014/10/24 02:11:10]

does this need an ifdef?
And could this go in ChromeDownloadManagerDelegate::SetDownloadManager instead?

[grt (UTC plus 2), 2014/10/30 19:07:35]

ifdef added.
Done. Is there an advantage other than keeping all safe_browsing integration
points in one file?

   }
 
   // Pass an empty delegate when constructing the DownloadUIController. The