Include both certificate chains in invalid cert reporting

net/cert/cert_verify_result.h

 // Copyright (c) 2011 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_VERIFY_RESULT_H_
 #define NET_CERT_CERT_VERIFY_RESULT_H_
 
 #include <vector>
 
 #include "base/memory/ref_counted.h"
@@ skipping 19 matching lines @@
   }
 
   // The certificate and chain that was constructed during verification.
   // Note that the though the verified certificate will match the originally
   // supplied certificate, the intermediate certificates stored within may
   // be substantially different. In the event of a verification failure, this
   // will contain the chain as supplied by the server. This may be NULL if
   // running within the sandbox.
   scoped_refptr<X509Certificate> verified_cert;
 
+  // The certificate and chain as sent by the server.
+  scoped_refptr<X509Certificate> unverified_server_cert;

[Ryan Sleevi, 2015/02/25 23:15:41]

This doesn't seem like it belongs in the CertVerifyResult, as it's a function of
the socket, not verification.

+
   // Bitmask of CERT_STATUS_* from net/base/cert_status_flags.h. Note that
   // these status flags apply to the certificate chain returned in
   // |verified_cert|, rather than the originally supplied certificate
   // chain.
   CertStatus cert_status;
 
   // Properties of the certificate chain.
   bool has_md2;
   bool has_md4;
   bool has_md5;
@@ skipping 15 matching lines @@
   bool is_issued_by_additional_trust_anchor;
 
   // True if a fallback to the common name was used when matching the host
   // name, rather than using the subjectAltName.
   bool common_name_fallback_used;
 };
 
 }  // namespace net
 
 #endif  // NET_CERT_CERT_VERIFY_RESULT_H_