Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(373)

Unified Diff: chrome/browser/safe_browsing/certificate_reporting_service_test_utils.cc

Issue 2503243003: Wire up CertificateReportingService to handle report uploads (Closed)
Patch Set: Rebase onto crrev/2543523002 Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/certificate_reporting_service_test_utils.cc
diff --git a/chrome/browser/safe_browsing/certificate_reporting_service_test_utils.cc b/chrome/browser/safe_browsing/certificate_reporting_service_test_utils.cc
index 6f9421a78ecdce68f69b8482e572e7c139638484..52a99a33416582343cf5ddb9e34ebf7438d60b33 100644
--- a/chrome/browser/safe_browsing/certificate_reporting_service_test_utils.cc
+++ b/chrome/browser/safe_browsing/certificate_reporting_service_test_utils.cc
@@ -205,4 +205,38 @@ void CertificateReportingServiceTestBase::ResumeDelayedRequest() {
url_request_interceptor_->Resume();
}
+void CertificateReportingServiceTestBase::CheckExpectedResults(
+ const ReportEventObserver& observer,
+ ObservedReportEvent expected_observed_event,
+ const std::set<int>& expected_successful_report_ids,
+ const std::set<int>& expected_failed_report_ids) {
+ switch (expected_observed_event) {
+ case EXPECT_NONE:
+ EXPECT_EQ(0u, observer.num_completed_attempts());
+ EXPECT_EQ(0u, observer.num_cancelled_attempts());
+ EXPECT_EQ(0u, observer.num_resets());
+ break;
+
+ case EXPECT_SEND_ATTEMPT_COMPLETED:
+ EXPECT_EQ(1u, observer.num_completed_attempts());
+ EXPECT_EQ(0u, observer.num_cancelled_attempts());
+ EXPECT_EQ(0u, observer.num_resets());
+ break;
+
+ case EXPECT_SEND_ATTEMPT_CANCELLED:
+ EXPECT_EQ(0u, observer.num_completed_attempts());
+ EXPECT_EQ(1u, observer.num_cancelled_attempts());
+ EXPECT_EQ(0u, observer.num_resets());
+ break;
+
+ case EXPECT_SERVICE_RESET:
+ EXPECT_EQ(0u, observer.num_completed_attempts());
+ EXPECT_EQ(0u, observer.num_cancelled_attempts());
+ EXPECT_EQ(1u, observer.num_resets());
+ break;
+ }
+ EXPECT_EQ(expected_successful_report_ids, observer.successful_report_ids());
+ EXPECT_EQ(expected_failed_report_ids, observer.failed_report_ids());
+}
+
} // namespace certificate_reporting_test_utils

Powered by Google App Engine
This is Rietveld 408576698