Expand UMA metrics for SSL (check history)

chrome/browser/ssl/ssl_blocking_page.h

 // Copyright (c) 2012 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_SSL_SSL_BLOCKING_PAGE_H_
 #define CHROME_BROWSER_SSL_SSL_BLOCKING_PAGE_H_
 
 #include <string>
 #include <vector>
 
 #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"
 
 namespace base {
 class DictionaryValue;
 }
 
 namespace content {
 class InterstitialPage;
@@ skipping 29 matching lines @@
   virtual void OverrideEntry(content::NavigationEntry* entry) OVERRIDE;
   virtual void OverrideRendererPrefs(
       content::RendererPreferences* prefs) OVERRIDE;
   virtual void OnProceed() OVERRIDE;
   virtual void OnDontProceed() OVERRIDE;
 
  private:
   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_;
   int cert_error_;
   base::TimeTicks display_start_time_;
   net::SSLInfo ssl_info_;
   GURL request_url_;
   // Could the user successfully override the error?
   bool overridable_;
   // Has the site requested strict enforcement of certificate errors?
   bool strict_enforcement_;
   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_;
 
   DISALLOW_COPY_AND_ASSIGN(SSLBlockingPage);
 };
 
 #endif  // CHROME_BROWSER_SSL_SSL_BLOCKING_PAGE_H_