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

Unified Diff: components/certificate_reporting/cert_logger.proto

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
« no previous file with comments | « no previous file | components/certificate_reporting/error_report.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/certificate_reporting/cert_logger.proto
diff --git a/components/certificate_reporting/cert_logger.proto b/components/certificate_reporting/cert_logger.proto
index e049841408afec43bdedcd97b1fa2da8dc23edb9..811d918a323c53f897060e02199934440d9534b6 100644
--- a/components/certificate_reporting/cert_logger.proto
+++ b/components/certificate_reporting/cert_logger.proto
@@ -67,6 +67,21 @@ message CertLoggerFeaturesInfo {
}
optional NetworkTimeQueryingInfo network_time_querying_info = 1;
+
+ // Records whether the Android AIA fetching feature is enabled.
+ //
+ // This is an enum rather than a boolean as a convenience to
+ // distinguish reports where fetching is disabled from reports that
+ // were sent before this field was present. (In other words, if it
+ // were a boolean, a value of false might mean that fetching was
+ // disabled, or it might mean that the browser version was older than
+ // when this field was added.)
+ enum AndroidAIAFetchingStatus {
+ ANDROID_AIA_FETCHING_UNKNOWN = 0;
+ ANDROID_AIA_FETCHING_ENABLED = 1;
+ ANDROID_AIA_FETCHING_DISABLED = 2;
+ }
+ optional AndroidAIAFetchingStatus android_aia_fetching_status = 2;
}
message CertLoggerRequest {
« no previous file with comments | « no previous file | components/certificate_reporting/error_report.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698