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

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

Issue 2365353004: Add Content-Type header to net::ReportSender reports (Closed)
Patch Set: make content type a required parameter to Send() Created 4 years, 3 months 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/permission_reporter_unittest.cc
diff --git a/chrome/browser/safe_browsing/permission_reporter_unittest.cc b/chrome/browser/safe_browsing/permission_reporter_unittest.cc
index 6f83885440d357c0aa0b85a3a0ac1a2cf96ede83..7a825cb4c2adacff16dc19305abca13eb7861cf8 100644
--- a/chrome/browser/safe_browsing/permission_reporter_unittest.cc
+++ b/chrome/browser/safe_browsing/permission_reporter_unittest.cc
@@ -118,6 +118,8 @@ TEST_F(PermissionReporterTest, SendReport) {
EXPECT_EQ(GURL(kPermissionActionReportingUploadUrl),
mock_report_sender_->latest_report_uri());
+ EXPECT_EQ("application/octet-stream",
+ mock_report_sender_->latest_content_type());
}
// Test that PermissionReporter::SendReport sends a serialized report string
@@ -153,6 +155,9 @@ TEST_F(PermissionReporterTest, SendReportWithFieldTrials) {
permission_reporter_->SendReport(BuildDummyReportInfo());
+ EXPECT_EQ("application/octet-stream",
+ mock_report_sender_->latest_content_type());
+
PermissionReport permission_report;
ASSERT_TRUE(
permission_report.ParseFromString(mock_report_sender_->latest_report()));

Powered by Google App Engine
This is Rietveld 408576698