Add support for the ExperimentalSwReporterEngine field trial.

chrome/browser/safe_browsing/srt_fetcher_win.h

 // Copyright 2015 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.
 
 #ifndef CHROME_BROWSER_SAFE_BROWSING_SRT_FETCHER_WIN_H_
 #define CHROME_BROWSER_SAFE_BROWSING_SRT_FETCHER_WIN_H_
 
 #include <limits.h>
+#include <stdint.h>
 
+#include <queue>
 #include <string>
 
 #include "base/callback_forward.h"
 #include "base/command_line.h"
 #include "base/memory/ref_counted.h"
 #include "base/version.h"
 
 namespace base {
 class FilePath;
 class TaskRunner;
@@ skipping 23 matching lines @@
 
 // Parameters used to invoke the sw_reporter component.
 struct SwReporterInvocation {
   base::CommandLine command_line;
 
   // Experimental versions of the reporter will write metrics to registry keys
   // ending in |suffix|. Those metrics should be copied to UMA histograms also
   // ending in |suffix|. For the canonical version, |suffix| will be empty.
   std::string suffix;
 
-  // The experimental sw_reporter never triggers the prompt, just reports
-  // results through UMA.
-  bool is_experimental = false;
+  // Flags to control optional behaviours. By default all are enabled;

[csharp, 2016/09/13 17:09:03]

What about disabling this all by default, to ensure they are only turned on when
they are meant to be (so a slight change to ComonentReady to set the flags value
for the default case)

[Joe Mason, 2016/09/13 19:24:55]

I think it's better to enable them by default because that's the expected common
case. Only the experimental reporter should be disabling any of these.

[Joe Mason, 2016/09/13 21:34:42]

csharp convinced me offline that we should set these to 0 by default, because if
anyone incorrectly adds a new use of SwReporterInvocation it's better for it not
to send logs and trigger the prompt.

+  // experimental versions of the reporter will turn off the behaviours that
+  // are not yet supported.
+  using Flags = uint32_t;
+  enum : Flags {
+    FLAG_LOG_TO_RAPPOR = 0x1,
+    FLAG_LOG_TO_PREFS = 0x2,
+    FLAG_TRIGGER_PROMPT = 0x4,
+  };
+  Flags flags = FLAG_LOG_TO_RAPPOR | FLAG_LOG_TO_PREFS | FLAG_TRIGGER_PROMPT;
 
   SwReporterInvocation();
 
   static SwReporterInvocation FromFilePath(const base::FilePath& exe_path);
   static SwReporterInvocation FromCommandLine(
       const base::CommandLine& command_line);
 
   bool operator==(const SwReporterInvocation& other) const;
 };
 
+using SwReporterQueue = std::queue<SwReporterInvocation>;
+
 // Tries to run the sw_reporter component, and then schedule the next try. If
 // called multiple times, then multiple sequences of trying to run will happen,
-// yet only one reporter will run per specified period (either
+// yet only one SwReporterQueue will actually run per specified period (either
 // |kDaysBetweenSuccessfulSwReporterRuns| or
-// |kDaysBetweenSwReporterRunsForPendingPrompt|) will actually happen.
-// |invocation| is the details of the SwReporter to execute, and |version| is
-// its version. The task runners are provided to allow tests to provide their
-// own.
-void RunSwReporter(const SwReporterInvocation& invocation,
-                   const base::Version& version,
-                   scoped_refptr<base::TaskRunner> main_thread_task_runner,
-                   scoped_refptr<base::TaskRunner> blocking_task_runner);
+// |kDaysBetweenSwReporterRunsForPendingPrompt|).
+//
+// Each "run" of the sw_reporter component may aggregate the results of several
+// executions of the tool with different command lines. |invocations| is the
+// queue of SwReporters to execute as a single "run". When a new try is
+// scheduled the entire queue is executed.
+//
+// |version| is the version of the tool that will run. The task runners are
+// provided to allow tests to provide their own.
+void RunSwReporters(const SwReporterQueue& invocations,
+                    const base::Version& version,
+                    scoped_refptr<base::TaskRunner> main_thread_task_runner,
+                    scoped_refptr<base::TaskRunner> blocking_task_runner);
 
 // Returns true iff Local State is successfully accessed and indicates the most
 // recent Reporter run terminated with an exit code indicating the presence of
 // UwS.
 bool ReporterFoundUws();
 
 // Returns true iff a valid registry key for the SRT Cleaner exists, and that
 // key is nonempty.
 // TODO(tmartino): Consider changing to check whether the user has recently
 // run the cleaner, rather than checking if they've run it at all.
@@ skipping 20 matching lines @@
 
 // Set a delegate for testing. The implementation will not take ownership of
 // |delegate| - it must remain valid until this function is called again to
 // reset the delegate. If |delegate| is nullptr, any previous delegate is
 // cleared.
 void SetSwReporterTestingDelegate(SwReporterTestingDelegate* delegate);
 
 }  // namespace safe_browsing
 
 #endif  // CHROME_BROWSER_SAFE_BROWSING_SRT_FETCHER_WIN_H_