Wire up download attribution enable finch experiment

chrome/browser/safe_browsing/safe_browsing_service.cc

Index: chrome/browser/safe_browsing/safe_browsing_service.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_service.cc b/chrome/browser/safe_browsing/safe_browsing_service.cc
index d03c2e4637e38292ab6b1501386d272bc71ee20b..f4254ab55acca0363ee132857c76a6dc4c14c6f9 100644
--- a/chrome/browser/safe_browsing/safe_browsing_service.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_service.cc
@@ -28,6 +28,7 @@
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/safe_browsing/ping_manager.h"
+#include "chrome/browser/safe_browsing/safe_browsing_navigation_observer_manager.h"
 #include "chrome/browser/safe_browsing/ui_manager.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
@@ -309,6 +310,11 @@ void SafeBrowsingService::Initialize() {
     database_manager_ = CreateDatabaseManager();
   }
 
+  if (base::FeatureList::IsEnabled(
+    SafeBrowsingNavigationObserverManager::kDownloadAttribution)) {

[Nathan Parker, 2016/12/20 01:01:33]

Should this go in the ServiceDelegateImpl, so it'll only get instantiated on
desktop?  I guess we will want it on Android soon.  And WebView? Maybe it's ok
to leave it here...

[Jialiu Lin, 2016/12/20 02:07:51]

Ack.

Eventually this navigation observer manager will be used for getting referrer
chain for interstitials too, then it'll get instantiated on both desktop and
android. That's why I feel it's probably better to leave it in
SafeBrowsingService.

+    navigation_observer_manager_ = new SafeBrowsingNavigationObserverManager();
+  }
+
   services_delegate_->Initialize();
   services_delegate_->InitializeCsdService(url_request_context_getter_.get());
 
@@ -393,6 +399,11 @@ SafeBrowsingService::database_manager() const {
   return enabled_v4_only_ ? v4_local_database_manager() : database_manager_;
 }
 
+const scoped_refptr<SafeBrowsingNavigationObserverManager>&
+SafeBrowsingService::navigation_observer_manager() const {
+  return navigation_observer_manager_;
+}
+
 SafeBrowsingProtocolManager* SafeBrowsingService::protocol_manager() const {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 #if defined(SAFE_BROWSING_DB_LOCAL)