Add Content-Type header to net::ReportSender reports

components/certificate_reporting/error_reporter_unittest.cc

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 6398c5fc180be55af13e7cda98dd511ba7ddf99e..6e83ca165be9601b1fb7226d19fe8a6bb36ad8e3 100644
--- a/components/certificate_reporting/error_reporter_unittest.cc
+++ b/components/certificate_reporting/error_reporter_unittest.cc
@@ -42,18 +42,24 @@ class MockCertificateReportSender : public net::ReportSender {
       : net::ReportSender(nullptr, DO_NOT_SEND_COOKIES) {}
   ~MockCertificateReportSender() override {}
 
-  void Send(const GURL& report_uri, const std::string& report) override {
+  void Send(const GURL& report_uri,

[Nathan Parker, 2016/09/26 23:53:25]

Since there are several tests reusing this pattern, could they reuse some code?

[estark, 2016/09/27 04:05:20]

Yeah, I think it might make sense to have a net::MockReportSender that they can
all use. I think I'll save it for a follow-up though because I might want to try
to merge this CL back to M54 (so I want it as small as possible).

+            base::StringPiece content_type,
+            base::StringPiece report) override {
     latest_report_uri_ = report_uri;
-    latest_report_ = report;
+    report.CopyToString(&latest_report_);
+    content_type.CopyToString(&latest_content_type_);
   }
 
   const GURL& latest_report_uri() { return latest_report_uri_; }
 
   const std::string& latest_report() { return latest_report_; }
 
+  const std::string& latest_content_type() { return latest_content_type_; }
+
  private:
   GURL latest_report_uri_;
   std::string latest_report_;
+  std::string latest_content_type_;
 
   DISALLOW_COPY_AND_ASSIGN(MockCertificateReportSender);
 };
@@ -121,6 +127,8 @@ TEST_F(ErrorReporterTest, ExtendedReportingSendReport) {
   http_reporter.SendExtendedReportingReport(kDummyReport);
 
   EXPECT_EQ(http_mock_report_sender->latest_report_uri(), http_url);
+  EXPECT_EQ("application/octet-stream",
+            http_mock_report_sender->latest_content_type());
 
   std::string uploaded_report;
   EncryptedCertLoggerRequest encrypted_request;