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

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

Issue 400323002: Refactor the captive portal code to move from the ssl_blocking_page class to the ssl_error_classific (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments Created 6 years, 5 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
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index a68a5e1786aa04c14fda056d50d6a7a5e4200424..35b17b69482072972031fbce0d18980ae27f10e5 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -9429,6 +9429,11 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="interstitial.ssl.captive_portal" enum="SSLCaptivePortal">
+ <owner>meacer@chromium.org</owner>
+ <summary>Record possible states of captive portals.</summary>
+</histogram>
+
<histogram name="interstitial.ssl.cause.nonoverridable"
enum="SSLNonAttackCauses">
<owner>felt@chromium.org</owner>
@@ -46591,6 +46596,31 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<int value="5" label="DEPRECATION_RAZE_FAILED">Raze failed.</int>
</enum>
+<enum name="SSLCaptivePortal" type="int">
+ <int value="0"
+ label="Chrome captive portal detection enabled
+ (CAPTIVE_PORTAL_DETECTION_ENABLED)"/>
+ <int value="1"
+ label="Chrome captive portal detection enabled on an overridable SSL
+ error page (CAPTIVE_PORTAL_DETECTION_ENABLED_OVERRIDABLE)"/>
+ <int value="2"
+ label="Received a captive portal result
+ (CAPTIVE_PORTAL_PROBE_COMPLETED)"/>
+ <int value="3"
+ label="Received a captive portal result on an overridable SSL error
+ page (CAPTIVE_PORTAL_PROBE_COMPLETED_OVERRIDABLE)"/>
+ <int value="4"
+ label="Received no response or Non-HTTP login page
+ (CAPTIVE_PORTAL_NO_RESPONSE)"/>
+ <int value="5"
+ label="Received no response or Non-HTTP login page on an overridable
+ SSL error page (CAPTIVE_PORTAL_NO_RESPONSE_OVERRIDABLE)"/>
+ <int value="6" label="Detected captive portal (CAPTIVE_PORTAL_DETECTED)"/>
+ <int value="7"
+ label="Detected captive portal on an overridable SSL error page
+ (CAPTIVE_PORTAL_DETECTED_OVERRIDABLE)"/>
+</enum>
+
<enum name="SSLCipherSuite" type="int">
<summary>SSL/TLS cipher suites from the IANA registry</summary>
<int value="0" label="TLS_NULL_WITH_NULL_NULL"/>

Powered by Google App Engine
This is Rietveld 408576698