Adds error handling support for the SwReporter launcher.

chrome/browser/safe_browsing/srt_fetcher_win.h

Index: chrome/browser/safe_browsing/srt_fetcher_win.h
diff --git a/chrome/browser/safe_browsing/srt_fetcher_win.h b/chrome/browser/safe_browsing/srt_fetcher_win.h
index 2d1ca858de6bbcbdeb014a1d7f3507f3e7f74af9..8f1582acd0fab28fbf83700ce9ceff8ba6cf7cbe 100644
--- a/chrome/browser/safe_browsing/srt_fetcher_win.h
+++ b/chrome/browser/safe_browsing/srt_fetcher_win.h
@@ -16,6 +16,7 @@
 #include "base/process/launch.h"
 #include "base/process/process.h"
 #include "base/time/time.h"
+#include "chrome/browser/safe_browsing/srt_chrome_prompt_impl.h"

[grt (UTC plus 2), 2017/04/25 07:06:40]

can you replace this with a forward decl for ChromePromptImpl and #include
"components/chrome_cleaner/public/interfaces/chrome_prompt.mojom.h"?

[ftirelo, 2017/04/25 22:24:05]

Done.

 
 namespace base {
 class FilePath;
@@ -28,7 +29,7 @@ class Browser;
 namespace safe_browsing {
 
 // A special exit code identifying a failure to run the reporter.
-const int kReporterFailureExitCode = INT_MAX;
+const int kReporterNotLaunchedExitCode = INT_MAX;
 
 // The number of days to wait before triggering another reporter run.
 const int kDaysBetweenSuccessfulSwReporterRuns = 7;
@@ -122,6 +123,19 @@ class SwReporterTestingDelegate {
 
   // A task runner used to spawn the reporter process (which blocks).
   virtual base::TaskRunner* BlockingTaskRunner() const = 0;
+
+  // Returns a ChromePromptImpl object that keeps track of specific actions
+  // during execution.
+  virtual std::unique_ptr<ChromePromptImpl> CreateChromePromptImpl(
+      chrome_cleaner::mojom::ChromePromptRequest request) = 0;
+
+  // Handles the case when the remote end has been closed, by performing the
+  // necessary cleanups if the prompt dialog is being shown to the user.
+  virtual void OnConnectionClosed() = 0;
+
+  // Handles the case when a mojo::ReportBadMessage has been explicitly
+  // reported.
+  virtual void OnConnectionError(const std::string& message) = 0;
 };
 
 // Set a delegate for testing. The implementation will not take ownership of