Add Rappor metrics for Safe Browsing and SSL interstitials.

chrome/browser/interstitials/security_interstitial_uma_helper.h

 // Copyright (c) 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.
 
 #ifndef CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_UMA_HELPER_H_
 #define CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_UMA_HELPER_H_
 
+#include <string>
+
 #include "base/task/cancelable_task_tracker.h"
 #include "base/time/time.h"
 #include "url/gurl.h"
 
 namespace content {
 class WebContents;
 }
 
 namespace extensions {
 class ExperienceSamplingEvent;
 }
 
 // Most of the security interstitials share a common layout and set of
 // choices. SecurityInterstitialUmaHelper is intended to help the security
-// interstitials record user choices in a common way via UMA histograms.
+// interstitials record user choices in a common way via UMA histograms
+// and RAPPOR metrics.
+// TODO(nparker): Rename *UmaHelper to *MetricsHelper
 class SecurityInterstitialUmaHelper {
  public:
   // These enums are used for histograms.  Don't reorder, delete, or insert
   // elements. New elements should be added at the end (right before the max).
   enum SecurityInterstitialDecision {
     SHOW,
     PROCEED,
     DONT_PROCEED,
     PROCEEDING_DISABLED,
     MAX_DECISION
   };
   enum SecurityInterstitialInteraction {
     TOTAL_VISITS,
     SHOW_ADVANCED,
     SHOW_PRIVACY_POLICY,
     SHOW_DIAGNOSTIC,
     SHOW_LEARN_MORE,
     RELOAD,
     OPEN_TIME_SETTINGS,
     MAX_INTERACTION
   };
 
+  // Args:
+  //   url: Full URL of page that triggered the interstitial

[felt, 2015/01/28 19:19:17]

Does it matter whether it's the full URL or just the origin?

[Nathan Parker, 2015/01/29 01:36:02]

Not currently, no, but I'd rather this code decide how to segment it.  That way
we could choose to start reporting the first path entry as well, w/o modifying
the callers.

Updated to make that more clear.

[felt, 2015/01/29 06:18:36]

The history manager lookup only works on a per-origin basis, so when you get
history callbacks it will only be useful for an origin. So be careful here, you
don't want to end up comparing hostnames to origins later.

[Nathan Parker, 2015/01/30 00:39:51]

Ah, interesting point.  So to accurately report (later) the isInHistory+site
tuple, we'll need to either make a history lookup on eTLD+1 ("Is this eTLD+1 in
my history?"), or report the full origin via Rappor.  The latter would be
potentially a large space for Rappor to match against (scheme + port + full
host), and we'd lose sight of any domains that are generating many unique
subdomains.  I'll go with eTLD+1, and add a way to (at least approximately)
determine if that matches the history.
 

+  //   uma_prefix: Histogram prefix.  e.g. "phishing"
+  //   rappor_prefix: Metric prefix.  e.g. "SafeBrowsing.Phishing"
+  //                  If empty, no data will be reported to Rappor.
+  //   sampling_event_name: Event name for Experience Sampling.
+  //                        e.g. "phishing_interstitial_"
   SecurityInterstitialUmaHelper(content::WebContents* web_contents,
                                 const GURL& url,
-                                const std::string& histogram_prefix,
+                                const std::string& uma_prefix,
+                                const std::string& rappor_prefix,
                                 const std::string& sampling_event_name);
   ~SecurityInterstitialUmaHelper();
 
-  // Record a user decision or interaction to the appropriate UMA histogram.
+  // Record a user decision or interaction to the appropriate UMA histogram
+  // and potentially in a RAPPOR metric.
   void RecordUserDecision(SecurityInterstitialDecision decision);
   void RecordUserInteraction(SecurityInterstitialInteraction interaction);
 
  private:
   // Used to query the HistoryService to see if the URL is in history.
   void OnGotHistoryCount(bool success, int num_visits, base::Time first_visit);
 
   content::WebContents* web_contents_;
   const GURL request_url_;
-  const std::string histogram_prefix_;
+  const std::string uma_prefix_;
+  const std::string rappor_prefix_;
   const std::string sampling_event_name_;
   int num_visits_;
   base::CancelableTaskTracker request_tracker_;
 #if defined(ENABLE_EXTENSIONS)
   scoped_ptr<extensions::ExperienceSamplingEvent> sampling_event_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(SecurityInterstitialUmaHelper);
 };
 
 #endif  // CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_UMA_HELPER_H_