Make sure to not remove global error twice.

chrome/browser/safe_browsing/srt_global_error_win.cc

 // Copyright 2014 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.
 
 #include "chrome/browser/safe_browsing/srt_global_error_win.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/files/file_util.h"
 #include "base/process/launch.h"
 #include "base/single_thread_task_runner.h"
 #include "base/thread_task_runner_handle.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/browser_process.h"
-#include "chrome/browser/safe_browsing/srt_field_trial_win.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/global_error/global_error_service.h"
 #include "components/component_updater/pref_names.h"
 #include "components/prefs/pref_service.h"
 #include "content/public/browser/browser_thread.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
@@ skipping 20 matching lines @@
 void MaybeExecuteSRTFromBlockingPool(
     const base::FilePath& downloaded_path,
     const scoped_refptr<SingleThreadTaskRunner>& task_runner,
     const base::Closure& success_callback,
     const base::Closure& failure_callback) {
   DCHECK(!downloaded_path.empty());
 
   if (base::PathExists(downloaded_path)) {
     base::FilePath executable_path(
         downloaded_path.ReplaceExtension(kExecutableExtension));
-    if (base::ReplaceFile(downloaded_path, executable_path, NULL)) {
+    if (base::ReplaceFile(downloaded_path, executable_path, nullptr)) {
       base::CommandLine srt_command_line(executable_path);
       srt_command_line.AppendSwitch(kChromePromptSwitch);
       base::Process srt_process(
           base::LaunchProcess(srt_command_line, base::LaunchOptions()));
       if (srt_process.IsValid()) {
         task_runner->PostTask(FROM_HERE, success_callback);
         return;
       }
     }
   }
@@ skipping 82 matching lines @@
   if (!interacted_) {
     // If user didn't interact with the bubble, it means they used the generic
     // close bubble button.
     RecordSRTPromptHistogram(SRT_PROMPT_CLOSED);
     g_browser_process->local_state()->SetBoolean(
         prefs::kSwReporterPendingPrompt, true);
   }
 }
 
 void SRTGlobalError::BubbleViewAcceptButtonPressed(Browser* browser) {
-  RecordSRTPromptHistogram(SRT_PROMPT_ACCEPTED);
-  interacted_ = true;
-  global_error_service_->RemoveGlobalError(this);
+  OnUserinteractionStarted(SRT_PROMPT_ACCEPTED);
   MaybeExecuteSRT();
 }
 
 void SRTGlobalError::BubbleViewCancelButtonPressed(Browser* browser) {
-  RecordSRTPromptHistogram(SRT_PROMPT_DENIED);
-  interacted_ = true;
-  global_error_service_->RemoveGlobalError(this);
-
+  OnUserinteractionStarted(SRT_PROMPT_DENIED);
   BrowserThread::PostBlockingPoolTask(
       FROM_HERE, base::Bind(&DeleteFilesFromBlockingPool, downloaded_path_));
   OnUserinteractionDone();
 }
 
 bool SRTGlobalError::ShouldCloseOnDeactivate() const {
   return false;
 }
 
 void SRTGlobalError::MaybeExecuteSRT() {
@@ skipping 21 matching lines @@
     browser->OpenURL(content::OpenURLParams(
         GURL(kSRTDownloadURL), content::Referrer(), NEW_FOREGROUND_TAB,
         ui::PAGE_TRANSITION_LINK, false));
   }
 
   BrowserThread::PostBlockingPoolTask(
       FROM_HERE, base::Bind(&DeleteFilesFromBlockingPool, downloaded_path_));
   OnUserinteractionDone();
 }
 
+void SRTGlobalError::OnUserinteractionStarted(
+    SRTPromptHistogramValue histogram_value) {
+  // This is for cases where the UI doesn't go away quickly enough and user
+  // might click on the button more than once, or more than one button.
+  if (interacted_)
+    return;
+  RecordSRTPromptHistogram(histogram_value);
+  interacted_ = true;
+  if (global_error_service_) {
+    global_error_service_->RemoveGlobalError(this);
+    global_error_service_ = nullptr;
+  }
+}
+
 void SRTGlobalError::OnUserinteractionDone() {
   DCHECK(interacted_);
   // Once the user interacted with the bubble, we can forget about any pending
   // prompt.
   g_browser_process->local_state()->SetBoolean(prefs::kSwReporterPendingPrompt,
                                                false);
   delete this;
 }
 
 }  // namespace safe_browsing