Removal of SignedCertificateTimestampStore

content/browser/loader/resource_request_info_impl.h

Index: content/browser/loader/resource_request_info_impl.h
diff --git a/content/browser/loader/resource_request_info_impl.h b/content/browser/loader/resource_request_info_impl.h
index 1e383504c0b26d41a9c388781d5774920cc9c8f3..6fffc7e18ddcff1d90f6ed92c61d6bc8845f1a87 100644
--- a/content/browser/loader/resource_request_info_impl.h
+++ b/content/browser/loader/resource_request_info_impl.h
@@ -64,6 +64,7 @@ class ResourceRequestInfoImpl : public ResourceRequestInfo,
       ResourceContext* context,
       base::WeakPtr<ResourceMessageFilter> filter,
       bool report_raw_headers,
+      bool send_extra_certificate_info,
       bool is_async,
       bool is_using_lofi,
       const std::string& original_headers);
@@ -91,6 +92,7 @@ class ResourceRequestInfoImpl : public ResourceRequestInfo,
   bool IsDownload() const override;
   bool IsUsingLoFi() const override;
   bool ShouldReportRawHeaders() const;
+  bool ShouldSendExtraCertificateInfo() const;

[Charlie Reis, 2016/05/11 22:30:23]

nit: This doesn't belong in this block if it's not an override from
ResourceRequestInfo.  Looks like we might want a blank line after IsUsingLoFi,
since ShouldReportRawHeaders isn't an override either.

Can you add comments above this to say what it's for, and how it differs from
ShouldReportRawHeaders?  I'm confused by how it's getting used.

[dwaxweiler, 2016/05/12 07:28:16]

Removed code again.

 
   CONTENT_EXPORT void AssociateWithRequest(net::URLRequest* request);
 
@@ -222,6 +224,7 @@ class ResourceRequestInfoImpl : public ResourceRequestInfo,
   // exits during a transfer.
   base::WeakPtr<ResourceMessageFilter> filter_;
   bool report_raw_headers_;
+  bool send_extra_certificate_info_;
   bool is_async_;
   bool is_using_lofi_;
   const std::string original_headers_;