Split captive portal metrics out of SSLErrorClassification

chrome/browser/interstitials/chrome_metrics_helper.cc

 // Copyright 2015 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 "chrome/browser/interstitials/chrome_metrics_helper.h"
 
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/history/history_service_factory.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ssl/captive_portal_metrics_recorder.h"
 #include "components/history/core/browser/history_service.h"
 #include "components/rappor/rappor_service.h"
 #include "content/public/browser/web_contents.h"
 
 #if defined(ENABLE_EXTENSIONS)
 #include "chrome/browser/extensions/api/experience_sampling_private/experience_sampling.h"
 #endif
 
 ChromeMetricsHelper::ChromeMetricsHelper(
     content::WebContents* web_contents,
     const GURL& request_url,
     const security_interstitials::MetricsHelper::ReportDetails settings,
     const std::string& sampling_event_name)
     : security_interstitials::MetricsHelper(
           request_url,
           settings,
           HistoryServiceFactory::GetForProfile(
               Profile::FromBrowserContext(web_contents->GetBrowserContext()),
               ServiceAccessType::EXPLICIT_ACCESS),
           g_browser_process->rappor_service()),
       web_contents_(web_contents),
       request_url_(request_url),
       sampling_event_name_(sampling_event_name) {
   DCHECK(!sampling_event_name_.empty());
 }
 
 ChromeMetricsHelper::~ChromeMetricsHelper() {}
 
+void ChromeMetricsHelper::StartRecordingCaptivePortalMetrics(bool overridable) {
+  captive_portal_recorder_.reset(
+      new CaptivePortalMetricsRecorder(web_contents_, overridable));
+}
+
+void ChromeMetricsHelper::RecordExtraShutdownMetrics() {
+  // The captive portal metrics should be recorded when the interstitial is
+  // closing (or destructing).
+  if (captive_portal_recorder_)
+    captive_portal_recorder_->RecordCaptivePortalUMAStatistics();
+}
+
 void ChromeMetricsHelper::RecordExtraUserDecisionMetrics(
     security_interstitials::MetricsHelper::Decision decision) {
 #if defined(ENABLE_EXTENSIONS)
   if (!sampling_event_.get()) {
     sampling_event_.reset(new extensions::ExperienceSamplingEvent(
         sampling_event_name_, request_url_,
         web_contents_->GetLastCommittedURL(),
         web_contents_->GetBrowserContext()));
   }
   switch (decision) {
@@ skipping 36 matching lines @@
     case TOTAL_VISITS:
     case SET_EXTENDED_REPORTING_ENABLED:
     case SET_EXTENDED_REPORTING_DISABLED:
     case EXTENDED_REPORTING_IS_ENABLED:
     case REPORT_PHISHING_ERROR:
     case MAX_INTERACTION:
       break;
   }
 #endif
 }