Expand UMA metrics for SSL (check history)

chrome/browser/ssl/ssl_blocking_page.h

Index: chrome/browser/ssl/ssl_blocking_page.h
diff --git a/chrome/browser/ssl/ssl_blocking_page.h b/chrome/browser/ssl/ssl_blocking_page.h
index af24412e2900e7bae3981013fb1aa187fc23ed70..cee8a309d6449ea6afc8eecf658f455d19cbee38 100644
--- a/chrome/browser/ssl/ssl_blocking_page.h
+++ b/chrome/browser/ssl/ssl_blocking_page.h
@@ -11,6 +11,7 @@
 #include "base/callback.h"
 #include "base/strings/string16.h"
 #include "base/time/time.h"
+#include "chrome/browser/history/history_service.h"
 #include "content/public/browser/interstitial_page_delegate.h"
 #include "net/ssl/ssl_info.h"
 #include "url/gurl.h"
@@ -60,6 +61,12 @@ class SSLBlockingPage : public content::InterstitialPageDelegate {
   void NotifyDenyCertificate();
   void NotifyAllowCertificate();
 
+  // Used to query the HistoryService to see if the URL is in history. For UMA.
+  void OnGotHistoryCount(HistoryService::Handle handle,
+                         bool success,
+                         int num_visits,
+                         base::Time first_visit);
+
   base::Callback<void(bool)> callback_;
 
   content::WebContents* web_contents_;
@@ -74,6 +81,9 @@ class SSLBlockingPage : public content::InterstitialPageDelegate {
   content::InterstitialPage* interstitial_page_;  // Owns us.
   // Is the hostname for an internal network?
   bool internal_;
+  // How many times is this same URL in history?
+  int num_visits_;
+  CancelableRequestConsumer request_consumer_;  // Used for getting num_visits_.

[palmer, 2013/08/09 17:38:14]

Nit: I'd stick with the comment-above-declaration style.

[felt, 2013/08/09 17:39:57]

Done.

 
   // For the FieldTrial: this contains the name of the condition.
   std::string trialCondition_;