Certificate Transparency: Adding finch and NetLog logging for EV certs

net/cert/cert_policy_enforcer.h

 // Copyright 2014 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_CERT_CERT_POLICY_ENFORCER_H
 #define NET_CERT_CERT_POLICY_ENFORCER_H
 
 #include <stddef.h>
 
 #include "net/base/net_export.h"
+#include "net/base/net_log.h"
 
 namespace net {
 
 namespace ct {
 
 struct CTVerifyResult;
 class EVCertsWhitelist;
 
 }  // namespace ct
 
 class X509Certificate;
 
 // Class for checking that a given certificate conforms to security-related
 // policies.
 class NET_EXPORT CertPolicyEnforcer {
  public:
   // Set the parameters for this policy enforcer:
   // |require_ct_for_ev| indicates whether Certificate Transparency presence
   // is required for EV certificates.
   explicit CertPolicyEnforcer(bool require_ct_for_ev);
   virtual ~CertPolicyEnforcer();
 
   // Returns true if the collection of SCTs for the given certificate
-  // conforms with the CT/EV policy.
+  // conforms with the CT/EV policy. If it is non-conforming, the certificate
+  // is logged to |net_log|.

[Ryan Sleevi, 2014/12/16 21:35:01]

This comment isn't accurate, since you always log to |net_log| ;)

[Eran Messeri, 2014/12/17 16:19:31]

Done - fixed.

   // |cert| is the certificate for which the SCTs apply.
   // |ct_result| must contain the result of verifying any SCTs associated with
   // |cert| prior to invoking this method.
   bool DoesConformToCTEVPolicy(X509Certificate* cert,
                                const ct::EVCertsWhitelist* ev_whitelist,
-                               const ct::CTVerifyResult& ct_result);
+                               const ct::CTVerifyResult& ct_result,
+                               const BoundNetLog& net_log);
 
  private:
-  bool IsCertificateInWhitelist(X509Certificate* cert,
-                                const ct::EVCertsWhitelist* ev_whitelist);
-
-  bool HasRequiredNumberOfSCTs(X509Certificate* cert,
-                               const ct::CTVerifyResult& ct_result);
-
   bool require_ct_for_ev_;
 };
 
 }  // namespace net
 
 #endif  // NET_CERT_CERT_POLICY_ENFORCER_H