diff options
| author | yosin <yosin@chromium.org> | 2015-09-29 03:52:09 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-09-29 10:52:51 +0000 |
| commit | d8f481307b6f23e561748fc88ec48bd032090602 (patch) | |
| tree | ef90d4512611c64c61c46260118fc73f2b0c4c6e /components/security_interstitials | |
| parent | 075bd08c1c9394cf34c3ea2820eb8527a7946d52 (diff) | |
| download | chromium_src-d8f481307b6f23e561748fc88ec48bd032090602.zip chromium_src-d8f481307b6f23e561748fc88ec48bd032090602.tar.gz chromium_src-d8f481307b6f23e561748fc88ec48bd032090602.tar.bz2 | |
Revert http://crrev.com/1365733005: Split captive portal metrics out of SSLErrorClassification
This reverts commit c0a91fac22f1d3ff3f8d052eb5dbffa70ef9835f.
android_clang_dbg_recipe bots failed compilation:
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_dbg_recipe/builds/126886
In file included from ../../chrome/browser/ssl/captive_portal_metrics_recorder.cc:5:
../../chrome/browser/ssl/captive_portal_metrics_recorder.h:43:8: error: private field 'captive_portal_detection_enabled_' is not used [-Werror,-Wunused-private-field]
bool captive_portal_detection_enabled_;
^
../../chrome/browser/ssl/captive_portal_metrics_recorder.h:45:8: error: private field 'captive_portal_probe_completed_' is not used [-Werror,-Wunused-private-field]
bool captive_portal_probe_completed_;
^
../../chrome/browser/ssl/captive_portal_metrics_recorder.h:47:8: error: private field 'captive_portal_no_response_' is not used [-Werror,-Wunused-private-field]
bool captive_portal_no_response_;
^
../../chrome/browser/ssl/captive_portal_metrics_recorder.h:48:8: error: private field 'captive_portal_detected_' is not used [-Werror,-Wunused-private-field]
bool captive_portal_detected_;
^
4 errors generated.
BUG=n/a
NOTRY=true
TBR=felt@chromium.org
Review URL: https://codereview.chromium.org/1380513003
Cr-Commit-Position: refs/heads/master@{#351279}
Diffstat (limited to 'components/security_interstitials')
| -rw-r--r-- | components/security_interstitials/core/metrics_helper.cc | 4 | ||||
| -rw-r--r-- | components/security_interstitials/core/metrics_helper.h | 5 |
2 files changed, 1 insertions, 8 deletions
diff --git a/components/security_interstitials/core/metrics_helper.cc b/components/security_interstitials/core/metrics_helper.cc index 7466562..da7002b 100644 --- a/components/security_interstitials/core/metrics_helper.cc +++ b/components/security_interstitials/core/metrics_helper.cc @@ -134,10 +134,6 @@ void MetricsHelper::RecordUserInteraction(Interaction interaction) { RecordExtraUserInteractionMetrics(interaction); } -void MetricsHelper::RecordShutdownMetrics() { - RecordExtraShutdownMetrics(); -} - int MetricsHelper::NumVisits() { return num_visits_; } diff --git a/components/security_interstitials/core/metrics_helper.h b/components/security_interstitials/core/metrics_helper.h index e610201..064a180 100644 --- a/components/security_interstitials/core/metrics_helper.h +++ b/components/security_interstitials/core/metrics_helper.h @@ -91,17 +91,14 @@ class MetricsHelper { // histogram and potentially in a RAPPOR metric. void RecordUserDecision(Decision decision); void RecordUserInteraction(Interaction interaction); - void RecordShutdownMetrics(); - // Number of times user visited this origin before. -1 means not-yet-set. int NumVisits(); protected: // Subclasses should implement any embedder-specific recording logic in these - // methods. They'll be invoked from the matching Record methods. + // methods. They'll be invoked from RecordUserDecision/Interaction. virtual void RecordExtraUserDecisionMetrics(Decision decision) = 0; virtual void RecordExtraUserInteractionMetrics(Interaction interaction) = 0; - virtual void RecordExtraShutdownMetrics() = 0; private: // Used to query the HistoryService to see if the URL is in history. It will |