| Index: components/security_interstitials/core/safe_browsing_error_ui.cc
|
| diff --git a/components/security_interstitials/core/safe_browsing_error_ui.cc b/components/security_interstitials/core/safe_browsing_error_ui.cc
|
| index a6a5bb6b16d649a4a8b599aff4d50f08ca4c2bd4..7c9a1e60b1e5aed3672baae376f568f8f8b30eed 100644
|
| --- a/components/security_interstitials/core/safe_browsing_error_ui.cc
|
| +++ b/components/security_interstitials/core/safe_browsing_error_ui.cc
|
| @@ -64,8 +64,7 @@ SafeBrowsingErrorUI::SafeBrowsingErrorUI(
|
| display_options_(display_options),
|
| app_locale_(app_locale),
|
| time_triggered_(time_triggered),
|
| - controller_(controller),
|
| - user_made_decision_(false) {
|
| + controller_(controller) {
|
| controller_->metrics_helper()->RecordUserDecision(MetricsHelper::SHOW);
|
| controller_->metrics_helper()->RecordUserInteraction(
|
| MetricsHelper::TOTAL_VISITS);
|
| @@ -75,12 +74,6 @@ SafeBrowsingErrorUI::SafeBrowsingErrorUI(
|
| }
|
|
|
| SafeBrowsingErrorUI::~SafeBrowsingErrorUI() {
|
| - // If the page is closing without an explicit decision, record it as not
|
| - // proceeding.
|
| - if (!user_made_decision_) {
|
| - controller_->metrics_helper()->RecordUserDecision(
|
| - MetricsHelper::DONT_PROCEED);
|
| - }
|
| controller_->metrics_helper()->RecordShutdownMetrics();
|
| }
|
|
|
| @@ -128,7 +121,6 @@ void SafeBrowsingErrorUI::HandleCommand(SecurityInterstitialCommands command) {
|
| controller_->metrics_helper()->RecordUserDecision(
|
| MetricsHelper::PROCEED);
|
| controller_->Proceed();
|
| - user_made_decision_ = true;
|
| break;
|
| }
|
| }
|
| @@ -147,7 +139,6 @@ void SafeBrowsingErrorUI::HandleCommand(SecurityInterstitialCommands command) {
|
| // commits.
|
| controller_->GoBackAfterNavigationCommitted();
|
| }
|
| - user_made_decision_ = true;
|
| break;
|
| }
|
| case CMD_DO_REPORT: {
|
|
|