Send safe browsing ThreatDetails to track download CTR when user tries to recover blocked downloads

chrome/browser/safe_browsing/safe_browsing_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/safe_browsing/safe_browsing_service.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 627 matching lines @@
 
   state_callback_list_.Notify();
 
 #if defined(FULL_SAFE_BROWSING)
   if (csd_service_)
     csd_service_->SetEnabledAndRefreshState(enable);
   if (download_service_)
     download_service_->SetEnabled(enable);
 #endif
 }
+
+void SafeBrowsingService::SendDownloadRecoveryReport(

[Nathan Parker, 2015/11/14 01:04:27]

How about moving the thread-hopping to the caller?  That would mean one less
function, and simpler function naming.

[Jialiu Lin, 2015/11/16 19:30:14]

Actually, it is more convenient to do thread hopping on SafeBrowsingService,
otherwise, DownloadDangerousPromp might need to implement
base::RefCountedThreadSafe<> class to be thread-safe. 

+    const std::string& report) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  BrowserThread::PostTask(
+      BrowserThread::IO, FROM_HERE,
+      base::Bind(&SafeBrowsingService::OnSendDownloadRecoveryReport, this,
+                 report));
+}
+
+void SafeBrowsingService::OnSendDownloadRecoveryReport(
+    const std::string& report) {
+  LOG(ERROR) << " OnSendDownloadRecoveryReport: " << report;
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  if (ping_manager())
+    ping_manager()->ReportThreatDetails(report);
+}