Check the SRT logs upload time for an entire queue of SRT invocations at once.

chrome/browser/safe_browsing/srt_fetcher_browsertest_win.cc

 // 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.
 
 #include "chrome/browser/safe_browsing/srt_fetcher_win.h"
 
 #include <iterator>
 #include <memory>
 #include <set>
 
@@ skipping 579 matching lines @@
   // this run.
   EnableSBExtendedReporting();
   SetLastTimeSentReport(kDaysBetweenReporterLogsSent - 1);
   int64_t last_time_sent_logs = GetLastTimeSentReport();
   RunReporter();
   TestReporterLaunchCycle({base::FilePath()});
   ExpectLoggingSwitches(std::set<std::string>{/*expect no switches*/});
   EXPECT_EQ(last_time_sent_logs, GetLastTimeSentReport());
 }
 
+IN_PROC_BROWSER_TEST_F(SRTFetcherTest, ReporterLogging_MultipleLaunches) {
+  exit_code_to_report_ = kSwReporterNothingFound;
+  base::test::ScopedFeatureList scoped_feature_list;
+  EnableSBExtendedReporting();
+  SetLastTimeSentReport(kDaysBetweenReporterLogsSent + 3);
+
+  const base::FilePath path1(L"path1");
+  const base::FilePath path2(L"path2");
+  SwReporterQueue invocations;
+  for (auto path : {path1, path2}) {

[Sorin Jianu, 2017/02/06 18:31:44]

do we want to use auto& to avoid a copy here?

[Joe Mason, 2017/02/06 18:50:52]

Good idea. Done.

+    auto invocation = SwReporterInvocation::FromFilePath(path);
+    invocation.supported_behaviours =
+        SwReporterInvocation::BEHAVIOUR_ALLOW_SEND_REPORTER_LOGS;
+    invocations.push(invocation);
+  }
+  RunReporterQueue(invocations);
+
+  // SBER is enabled and last time logs were sent was more than
+  // |kDaysBetweenReporterLogsSent| day ago, so we should send logs in this run.
+  {
+    SCOPED_TRACE("first launch");
+    TestPartialLaunchCycle({path1});
+    ExpectLoggingSwitches(std::set<std::string>(std::begin(kExpectedSwitches),
+                                                std::end(kExpectedSwitches)));
+    ExpectLastReportSentInTheLastHour();
+  }
+
+  // Logs should also be sent for the next run, even though LastTimeSentReport
+  // is now recent, because the run is part of the same set of invocations.
+  {
+    SCOPED_TRACE("second launch");
+    TestReporterLaunchCycle({path2});
+    ExpectLoggingSwitches(std::set<std::string>(std::begin(kExpectedSwitches),
+                                                std::end(kExpectedSwitches)));
+    ExpectLastReportSentInTheLastHour();
+  }
+}
+
 }  // namespace safe_browsing