| Index: chrome/browser/ssl/cert_report_helper.cc
|
| diff --git a/chrome/browser/ssl/cert_report_helper.cc b/chrome/browser/ssl/cert_report_helper.cc
|
| index 1c41bb4820754df112a5a5fd3476426bbaae7677..9eb4cfbd692f3545b92e5b4fc54333dbd4d5368a 100644
|
| --- a/chrome/browser/ssl/cert_report_helper.cc
|
| +++ b/chrome/browser/ssl/cert_report_helper.cc
|
| @@ -96,20 +96,26 @@ void CertReportHelper::PopulateExtendedReportingOption(
|
|
|
| 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_);
|
| @@ -125,6 +131,7 @@ void CertReportHelper::FinishCertCollection(
|
|
|
| if (!report.Serialize(&serialized_report)) {
|
| LOG(ERROR) << "Failed to serialize certificate report.";
|
| + ssl_cert_reporter_->OnDidNotSendReport();
|
| return;
|
| }
|
|
|
|
|