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

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 985993003: Certificate Transparency: Add UMA for whitelist status (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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:
Download patch
« net/cert/cert_policy_enforcer.cc ('K') | « net/cert/cert_policy_enforcer.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 4fd69481473431e2954fa9b5ccbb37c6646fafb8..3f6a943bb7a693922f721f5748a29a22de13d6a4 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -20212,6 +20212,19 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="Net.SSL_EVWhitelistValidityForNonCompliantCert" enum="Boolean">
jwd 2015/03/09 15:26:41 Can you use a more precise enum type, maybe Boolea
Eran Messeri 2015/03/10 21:16:35 Done - defined my own. Can use BooleanValid if you
+ <owner>eranm@chromium.org</owner>
+ <owner>rsleevi@chromium.org</owner>
+ <summary>
+ Whether the client holds a valid EV Certificates whitelist or not. Only
+ emitted when an EV cert that is not compliant with the Certificate
+ Transparency requirement encountered.
jwd 2015/03/09 15:26:41 nit: "... is encountered."
Eran Messeri 2015/03/10 21:16:35 Done.
+ This histogram is intended to be short-lived and help determine if EV
+ certificates are considered non-compliant because they are not whitelisted
+ or if the client does not hold a valid instance of the whitelist.
+ </summary>
+</histogram>
+
<histogram name="Net.SSLCertBlacklisted">
<owner>agl@chromium.org</owner>
<summary>
« net/cert/cert_policy_enforcer.cc ('K') | « net/cert/cert_policy_enforcer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698