Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(223)

Unified Diff: components/certificate_reporting/error_report.cc

Issue 2682733003: Add AIA fetching feature info to certificate reports (Closed)
Patch Set: Add UMA statistic for AIA fetches Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/certificate_reporting/error_report.cc
diff --git a/components/certificate_reporting/error_report.cc b/components/certificate_reporting/error_report.cc
index 86d345f4cc0f5014991175bebaa111a3fc1eb657..dc3eb53cd08b9fd5843d93a2d88a01b5ab0c90f9 100644
--- a/components/certificate_reporting/error_report.cc
+++ b/components/certificate_reporting/error_report.cc
@@ -15,6 +15,10 @@
#include "net/cert/x509_certificate.h"
#include "net/ssl/ssl_info.h"
+#if defined(OS_ANDROID)
+#include "net/cert/cert_verify_proc_android.h"
+#endif
+
using network_time::NetworkTimeTracker;
namespace certificate_reporting {
@@ -85,6 +89,15 @@ ErrorReport::ErrorReport(const std::string& hostname,
cert_report_->set_is_issued_by_known_root(ssl_info.is_issued_by_known_root);
AddCertStatusToReportErrors(ssl_info.cert_status, cert_report_.get());
+
+#if defined(OS_ANDROID)
+ CertLoggerFeaturesInfo* features_info = cert_report_->mutable_features_info();
+ features_info->set_android_aia_fetching_status(
+ base::FeatureList::IsEnabled(
+ net::CertVerifyProcAndroid::kAIAFetchingFeature)
+ ? CertLoggerFeaturesInfo::ANDROID_AIA_FETCHING_ENABLED
+ : CertLoggerFeaturesInfo::ANDROID_AIA_FETCHING_DISABLED);
+#endif
}
ErrorReport::~ErrorReport() {}
« no previous file with comments | « components/certificate_reporting/cert_logger.proto ('k') | components/certificate_reporting/error_report_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698