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

chrome/browser/download/download_service.cc

 // 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.
 
 #include "chrome/browser/download/download_service.h"
 
 #include "base/callback.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/download/chrome_download_manager_delegate.h"
 #include "chrome/browser/download/download_history.h"
 #include "chrome/browser/download/download_service_factory.h"
 #include "chrome/browser/download/download_status_updater.h"
 #include "chrome/browser/download/download_ui_controller.h"
 #include "chrome/browser/history/history_service.h"
 #include "chrome/browser/history/history_service_factory.h"
 #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)
 #include "chrome/browser/extensions/api/downloads/downloads_api.h"
 #endif
 
 using content::BrowserContext;
 using content::DownloadManager;
 using content::DownloadManagerDelegate;
 
@@ skipping 28 matching lines @@
 
   if (!profile_->IsOffTheRecord()) {
     HistoryService* history = HistoryServiceFactory::GetForProfile(
         profile_, Profile::EXPLICIT_ACCESS);
     history->GetNextDownloadId(
         manager_delegate_->GetDownloadIdReceiverCallback());
     download_history_.reset(new DownloadHistory(
         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
   // default delegate does all the notifications we need.
   download_ui_.reset(new DownloadUIController(
       manager, scoped_ptr<DownloadUIController::Delegate>()));
 
   // Include this download manager in the set monitored by the
   // global status updater.
   g_browser_process->download_status_updater()->AddManager(manager);
@@ skipping 94 matching lines @@
     // code) when the DownloadManager is shutting down. So we shut it down
     // manually earlier. See http://crbug.com/131692
     BrowserContext::GetDownloadManager(profile_)->Shutdown();
   }
 #if defined(ENABLE_EXTENSIONS)
   extension_event_router_.reset();
 #endif
   manager_delegate_.reset();
   download_history_.reset();
 }