Implement main CertificateReportingService code and add unit tests.

chrome/browser/ssl/cert_report_helper.cc

 // Copyright 2015 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/ssl/cert_report_helper.h"
 
 #include <utility>
 
 #include "base/logging.h"
 #include "base/metrics/field_trial.h"
@@ skipping 78 matching lines @@
       security_interstitials::kOptInLink,
       l10n_util::GetStringFUTF16(safe_browsing::ChooseOptInTextResource(
                                      *GetProfile(web_contents_)->GetPrefs(),
                                      IDS_SAFE_BROWSING_MALWARE_REPORTING_AGREE,
                                      IDS_SAFE_BROWSING_SCOUT_REPORTING_AGREE),
                                  base::UTF8ToUTF16(privacy_link)));
 }
 
 void CertReportHelper::FinishCertCollection(
     certificate_reporting::ErrorReport::ProceedDecision user_proceeded) {
-  if (!ShouldShowCertificateReporterCheckbox())
+  if (!ShouldShowCertificateReporterCheckbox()) {
+    ssl_cert_reporter_->OnDidNotSendReport();
     return;
+  }
 
   if (!safe_browsing::IsExtendedReportingEnabled(
-          *GetProfile(web_contents_)->GetPrefs()))
+          *GetProfile(web_contents_)->GetPrefs())) {
+    ssl_cert_reporter_->OnDidNotSendReport();
     return;
+  }
 
   if (metrics_helper_) {
     metrics_helper_->RecordUserInteraction(
         security_interstitials::MetricsHelper::EXTENDED_REPORTING_IS_ENABLED);
   }
 
-  if (!ShouldReportCertificateError())
+  if (!ShouldReportCertificateError()) {
+    ssl_cert_reporter_->OnDidNotSendReport();
     return;
+  }
 
   std::string serialized_report;
   certificate_reporting::ErrorReport report(request_url_.host(), ssl_info_);
 
   report.AddNetworkTimeInfo(g_browser_process->network_time_tracker());
 
   report.SetInterstitialInfo(
       interstitial_reason_, user_proceeded,
       overridable_
           ? certificate_reporting::ErrorReport::INTERSTITIAL_OVERRIDABLE
           : certificate_reporting::ErrorReport::INTERSTITIAL_NOT_OVERRIDABLE,
       interstitial_time_);
 
   if (!report.Serialize(&serialized_report)) {
     LOG(ERROR) << "Failed to serialize certificate report.";

[Jialiu Lin, 2016/11/30 22:18:07]

maybe remove this LOG(ERROR)? 

[meacer, 2016/11/30 23:39:40]

estark: I think this is your LOG. Is it okay to remove it?

+    ssl_cert_reporter_->OnDidNotSendReport();
     return;
   }
 
   ssl_cert_reporter_->ReportInvalidCertificateChain(serialized_report);
 }
 
 void CertReportHelper::SetSSLCertReporterForTesting(
     std::unique_ptr<SSLCertReporter> ssl_cert_reporter) {
   ssl_cert_reporter_ = std::move(ssl_cert_reporter);
 }
@@ skipping 22 matching lines @@
       if (sendingThreshold >= 0.0 && sendingThreshold <= 1.0)
         return base::RandDouble() <= sendingThreshold;
     }
   }
   return false;
 }
 
 bool CertReportHelper::IsPrefEnabled(const char* pref) {
   return GetProfile(web_contents_)->GetPrefs()->GetBoolean(pref);
 }