Remove unused CookiePreferences from report sender and never send cookies

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 076049dd916465846a0b63a8dfad64fbf61db82e..1a3548dd4d6b08875b13dd77e00aea2917f29de5 100644
--- a/components/certificate_reporting/error_reporter_unittest.cc
+++ b/components/certificate_reporting/error_reporter_unittest.cc
@@ -46,8 +46,7 @@ void SuccessCallback(bool* called) {
 // sent.
 class MockCertificateReportSender : public net::ReportSender {
  public:
-  MockCertificateReportSender()
-      : net::ReportSender(nullptr, DO_NOT_SEND_COOKIES) {}
+  MockCertificateReportSender() : net::ReportSender(nullptr) {}
   ~MockCertificateReportSender() override {}
 
   void Send(
@@ -170,10 +169,9 @@ TEST_F(ErrorReporterTest, ErroredRequestCallsCallback) {
   context.set_network_delegate(&test_delegate);
   context.Init();
 
-  GURL report_uri(
+  const GURL report_uri(
       net::URLRequestFailedJob::GetMockHttpUrl(net::ERR_CONNECTION_FAILED));
-  ErrorReporter reporter(&context, report_uri,
-                         net::ReportSender::DO_NOT_SEND_COOKIES);
+  ErrorReporter reporter(&context, report_uri);
 
   bool error_callback_called = false;
   bool success_callback_called = false;
@@ -197,9 +195,9 @@ TEST_F(ErrorReporterTest, SuccessfulRequestCallsCallback) {
   context.set_network_delegate(&test_delegate);
   context.Init();
 
-  GURL report_uri(net::URLRequestMockDataJob::GetMockHttpUrl("some data", 1));
-  ErrorReporter reporter(&context, report_uri,
-                         net::ReportSender::DO_NOT_SEND_COOKIES);
+  const GURL report_uri(
+      net::URLRequestMockDataJob::GetMockHttpUrl("some data", 1));
+  ErrorReporter reporter(&context, report_uri);
 
   bool error_callback_called = false;
   bool success_callback_called = false;