Add UMA histogram to count hard revokes of user certificate error decisions.

chrome/browser/ui/website_settings/website_settings.cc

Index: chrome/browser/ui/website_settings/website_settings.cc
diff --git a/chrome/browser/ui/website_settings/website_settings.cc b/chrome/browser/ui/website_settings/website_settings.cc
index bdb6dd098c6757e2b0cdea4f3ba38ca1e977880f..8c0819103124debe8001a5a2658b52e332a3569b 100644
--- a/chrome/browser/ui/website_settings/website_settings.cc
+++ b/chrome/browser/ui/website_settings/website_settings.cc
@@ -61,6 +61,13 @@ using content::BrowserThread;
 
 namespace {
 
+// Events for UMA. Do not reorder or change!
+enum SSLCertificateDecisionsDidRevoke {
+  USER_CERT_DECISIONS_NOT_REVOKED = 0,
+  USER_CERT_DECISIONS_REVOKED,
+  END_OF_SSL_CERTIFICATE_DECISIONS_DID_REVOKE_ENUM
+};
+
 // The list of content settings types to display on the Website Settings UI.
 ContentSettingsType kPermissionType[] = {
   CONTENT_SETTINGS_TYPE_IMAGES,
@@ -174,7 +181,8 @@ WebsiteSettings::WebsiteSettings(
       cert_store_(cert_store),
       content_settings_(profile->GetHostContentSettingsMap()),
       chrome_ssl_host_state_delegate_(
-          ChromeSSLHostStateDelegateFactory::GetForProfile(profile)) {
+          ChromeSSLHostStateDelegateFactory::GetForProfile(profile)),
+      did_revoke_user_ssl_decisions_(false) {
   Init(profile, url, ssl);
 
   HistoryService* history_service = HistoryServiceFactory::GetForProfile(
@@ -321,6 +329,20 @@ void WebsiteSettings::OnSiteDataAccessed() {
 void WebsiteSettings::OnUIClosing() {
   if (show_info_bar_)
     WebsiteSettingsInfoBarDelegate::Create(infobar_service_);
+
+  SSLCertificateDecisionsDidRevoke user_decision =
+      did_revoke_user_ssl_decisions_ ? USER_CERT_DECISIONS_REVOKED
+                                     : USER_CERT_DECISIONS_NOT_REVOKED;

[Peter Kasting, 2014/08/22 22:44:00]

Nit: clang-format error; wrap like:

      did_revoke_user_ssl_decisions_ ?
          USER_CERT_DECISIONS_REVOKED : USER_CERT_DECISIONS_NOT_REVOKED;

[jww, 2014/08/22 22:56:39]

I'm going to push back on this. This instance was not a clang-format decision;
this matches what is done throughout the rest of this file (see lines 108 - 119,
for example).

[Peter Kasting, 2014/08/22 23:02:22]

It's a nit, so you can override me.  And being consistent with the file is a
fine reason too.  Historically, this style never existed in Chromium until
clang-format introduced it, and despite some discussions that I thought ended
with someone agreeing to fix clang-format not to do it, it still seems to crop
up frequently, so I do my best to prevent it spreading.

+
+  UMA_HISTOGRAM_ENUMERATION("interstitial.ssl.did_user_revoke_decisions",
+                            user_decision,
+                            END_OF_SSL_CERTIFICATE_DECISIONS_DID_REVOKE_ENUM);
+}
+
+void WebsiteSettings::OnSSLDecisionsRevokeButtonPressed() {
+  DCHECK(chrome_ssl_host_state_delegate_);
+  chrome_ssl_host_state_delegate_->RevokeUserDecisionsHard(site_url().host());
+  did_revoke_user_ssl_decisions_ = true;
 }
 
 void WebsiteSettings::Init(Profile* profile,