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

Unified Diff: components/certificate_reporting/error_reporter_unittest.cc

Issue 2483993002: Add report_id to ErrorReporter interface. (Closed)
Patch Set: Created 4 years, 1 month 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: components/certificate_reporting/error_reporter_unittest.cc
diff --git a/components/certificate_reporting/error_reporter_unittest.cc b/components/certificate_reporting/error_reporter_unittest.cc
index 20e768638ed7df22af2c23508f9cb50e40fbfa9f..c06f9844d521a46a12400f79ea6e32c1644aac7e 100644
--- a/components/certificate_reporting/error_reporter_unittest.cc
+++ b/components/certificate_reporting/error_reporter_unittest.cc
@@ -120,7 +120,7 @@ TEST_F(ErrorReporterTest, ExtendedReportingSendReport) {
ErrorReporter https_reporter(https_url, server_public_key_,
kServerPublicKeyTestVersion,
base::WrapUnique(mock_report_sender));
- https_reporter.SendExtendedReportingReport(kDummyReport);
+ https_reporter.SendExtendedReportingReport(kDummyReport, 0);
EXPECT_EQ(mock_report_sender->latest_report_uri(), https_url);
EXPECT_EQ(mock_report_sender->latest_report(), kDummyReport);
@@ -131,7 +131,7 @@ TEST_F(ErrorReporterTest, ExtendedReportingSendReport) {
ErrorReporter http_reporter(http_url, server_public_key_,
kServerPublicKeyTestVersion,
base::WrapUnique(http_mock_report_sender));
- http_reporter.SendExtendedReportingReport(kDummyReport);
+ http_reporter.SendExtendedReportingReport(kDummyReport, 0);
EXPECT_EQ(http_mock_report_sender->latest_report_uri(), http_url);
EXPECT_EQ("application/octet-stream",
@@ -170,7 +170,7 @@ TEST_F(ErrorReporterTest, UMAOnFailure) {
net::URLRequestFailedJob::GetMockHttpUrl(net::ERR_CONNECTION_FAILED));
ErrorReporter reporter(&context, report_uri,
net::ReportSender::DO_NOT_SEND_COOKIES);
- reporter.SendExtendedReportingReport(kDummyReport);
+ reporter.SendExtendedReportingReport(kDummyReport, 0);
run_loop.Run();
histograms.ExpectTotalCount(kFailureHistogramName, 1);

Powered by Google App Engine
This is Rietveld 408576698