diff options
author | abarth@chromium.org <abarth@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-04-26 23:54:41 +0000 |
---|---|---|
committer | abarth@chromium.org <abarth@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-04-26 23:54:41 +0000 |
commit | 1d2d22b12a66fce2615da7b271af5666a66ce70f (patch) | |
tree | da4f8ec6dce86a1d2551eb78c9ca56402b89809c /chrome/browser/ssl/ssl_blocking_page.cc | |
parent | 7b0aeeaf4647dca4d00c954ec13e8f20f587358e (diff) | |
download | chromium_src-1d2d22b12a66fce2615da7b271af5666a66ce70f.zip chromium_src-1d2d22b12a66fce2615da7b271af5666a66ce70f.tar.gz chromium_src-1d2d22b12a66fce2615da7b271af5666a66ce70f.tar.bz2 |
Revert 14569
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14573 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ssl/ssl_blocking_page.cc')
-rw-r--r-- | chrome/browser/ssl/ssl_blocking_page.cc | 23 |
1 files changed, 1 insertions, 22 deletions
diff --git a/chrome/browser/ssl/ssl_blocking_page.cc b/chrome/browser/ssl/ssl_blocking_page.cc index a34c86a..ce9d9da 100644 --- a/chrome/browser/ssl/ssl_blocking_page.cc +++ b/chrome/browser/ssl/ssl_blocking_page.cc @@ -4,7 +4,6 @@ #include "chrome/browser/ssl/ssl_blocking_page.h" -#include "base/histogram.h" #include "base/string_piece.h" #include "base/values.h" #include "chrome/browser/browser.h" @@ -23,22 +22,6 @@ #include "grit/browser_resources.h" #include "grit/generated_resources.h" -namespace { - -enum SSLBlockingPageEvent { - SHOW, - PROCEED, - DONT_PROCEED, -}; - -void RecordSSLBlockingPageStats(SSLBlockingPageEvent event) { - static LinearHistogram histogram("interstial.ssl", 0, 2, 4); - histogram.SetFlags(kUmaTargetedHistogramFlag); - histogram.Add(event); -} - -} // namespace - // Note that we always create a navigation entry with SSL errors. // No error happening loading a sub-resource triggers an interstitial so far. SSLBlockingPage::SSLBlockingPage(SSLManager::CertError* error, @@ -47,7 +30,6 @@ SSLBlockingPage::SSLBlockingPage(SSLManager::CertError* error, error_(error), delegate_(delegate), delegate_has_been_notified_(false) { - RecordSSLBlockingPageStats(SHOW); } SSLBlockingPage::~SSLBlockingPage() { @@ -112,8 +94,6 @@ void SSLBlockingPage::CommandReceived(const std::string& command) { } void SSLBlockingPage::Proceed() { - RecordSSLBlockingPageStats(PROCEED); - // Accepting the certificate resumes the loading of the page. NotifyAllowCertificate(); @@ -122,12 +102,11 @@ void SSLBlockingPage::Proceed() { } void SSLBlockingPage::DontProceed() { - RecordSSLBlockingPageStats(DONT_PROCEED); - NotifyDenyCertificate(); InterstitialPage::DontProceed(); } + void SSLBlockingPage::NotifyDenyCertificate() { DCHECK(!delegate_has_been_notified_); |