Remove redundancy in security interstitial UMA logic

chrome/browser/ssl/ssl_error_classification.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <vector>
 
 #include "chrome/browser/ssl/ssl_error_classification.h"
 
 #include "base/build_time.h"
 #include "base/metrics/field_trial.h"
@@ skipping 279 matching lines @@
   if (IsHostnameNonUniqueOrDotless(request_url_.HostNoBrackets()))
     severity_authority_score += kClientWeight * kPrivateURLWeight;
 
   RecordSSLInterstitialSeverityScore(severity_authority_score, cert_error_);
 }
 
 void SSLErrorClassification::RecordUMAStatistics(
     bool overridable) const {
   SSLErrorInfo::ErrorType type =
       SSLErrorInfo::NetErrorToErrorType(cert_error_);
+  UMA_HISTOGRAM_ENUMERATION(
+      "interstitial.ssl_error_type", type, SSLErrorInfo::END_OF_ENUM);
   switch (type) {
     case SSLErrorInfo::CERT_DATE_INVALID: {
       if (IsUserClockInThePast(base::Time::NowFromSystemTime()))
         RecordSSLInterstitialCause(overridable, CLOCK_PAST);
       if (IsUserClockInTheFuture(base::Time::NowFromSystemTime()))
         RecordSSLInterstitialCause(overridable, CLOCK_FUTURE);
       break;
     }
     case SSLErrorInfo::CERT_COMMON_NAME_INVALID: {
       std::string host_name = request_url_.host();
@@ skipping 336 matching lines @@
     // sure we don't clear the captive protal flag, since the interstitial was
     // potentially caused by the captive portal.
     captive_portal_detected_ = captive_portal_detected_ ||
         (results->result == captive_portal::RESULT_BEHIND_CAPTIVE_PORTAL);
     // Also keep track of non-HTTP portals and error cases.
     captive_portal_no_response_ = captive_portal_no_response_ ||
         (results->result == captive_portal::RESULT_NO_RESPONSE);
   }
 #endif
 }