Add net::CertificateReportSender for handling cert report sending

net/http/certificate_report_sender_impl.h

Index: net/http/certificate_report_sender_impl.h
diff --git a/net/http/certificate_report_sender_impl.h b/net/http/certificate_report_sender_impl.h
new file mode 100644
index 0000000000000000000000000000000000000000..18d1dba5a7950059861881cd12866685520ad139
--- /dev/null
+++ b/net/http/certificate_report_sender_impl.h
@@ -0,0 +1,69 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef NET_HTTP_CERTIFICATE_REPORT_SENDER_IMPL_H_
+#define NET_HTTP_CERTIFICATE_REPORT_SENDER_IMPL_H_
+
+#include <set>
+#include <string>
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "net/http/certificate_report_sender.h"
+#include "net/url_request/url_request.h"

[davidben, 2015/07/23 00:09:42]

Nothing else in net/http includes anything from net/url_request. Perhaps this
wants to be in net/url_request while the interface stays in net/http so
TransportSecurityState can get at it.

[estark, 2015/07/23 02:41:21]

Done.

+
+class GURL;
+
+namespace net {
+
+class URLRequestContext;
+
+// An implementation of CertificateReportSender that sends reports with
+// a given URLRequestContext. It owns inflight requests and cleans them
+// up when necessary.
+class NET_EXPORT CertificateReportSenderImpl : public URLRequest::Delegate,
+                                               public CertificateReportSender {
+ public:
+  // Represents whether or not to send cookies along with reports.
+  enum CookiesPreference { SEND_COOKIES, DO_NOT_SEND_COOKIES };
+
+  // Constructs a CertificateReportSender that sends reports with the
+  // given |request_context| and includes or excludes cookies based on
+  // |cookies_preference|. Ownership of |request_context| is not
+  // transferred, so it must outlive the CertificateReportSenderImpl.
+  CertificateReportSenderImpl(URLRequestContext* request_context,
+                              CookiesPreference cookies_preference);
+
+  ~CertificateReportSenderImpl() override;
+
+  // CertificateReportSender overrides
+  void Send(const GURL& report_uri, const std::string& report) override;
+
+  // net::URLRequest::Delegate
+  void OnResponseStarted(URLRequest* request) override;
+  void OnReadCompleted(URLRequest* request, int bytes_read) override;
+
+ private:
+  // Creates a URLRequest with which to send a certificate report to the
+  // server.
+  virtual scoped_ptr<URLRequest> CreateURLRequest(
+      net::URLRequestContext* context,
+      const GURL& report_uri);
+
+  // Performs post-report cleanup.
+  void RequestComplete(net::URLRequest* request);
+
+  net::URLRequestContext* const request_context_;
+
+  CookiesPreference cookies_preference_;
+
+  // Owns the contained requests.
+  std::set<URLRequest*> inflight_requests_;
+
+  DISALLOW_COPY_AND_ASSIGN(CertificateReportSenderImpl);
+};
+
+}  // namespace net
+
+#endif  // NET_HTTP_CERTIFICATE_REPORT_SENDER_H_