Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1249)

Unified Diff: net/quic/crypto/proof_verifier_chromium.h

Issue 436753002: Extend ProofVerifierChromium and ProofVerifyDetailsChromium. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove an unneeded member of ProofVerifierChromium Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/quic/crypto/proof_verifier_chromium.h
diff --git a/net/quic/crypto/proof_verifier_chromium.h b/net/quic/crypto/proof_verifier_chromium.h
index 5e6ba99447cab8c3a8df611a97bd801d0d4df7e4..2c64808c5f94b2ae5d0e3acfe047c4ea5998f470 100644
--- a/net/quic/crypto/proof_verifier_chromium.h
+++ b/net/quic/crypto/proof_verifier_chromium.h
@@ -21,6 +21,7 @@
namespace net {
class CertVerifier;
+class TransportSecurityState;
// ProofVerifyDetailsChromium is the implementation-specific information that a
// ProofVerifierChromium returns about a certificate verification.
@@ -32,6 +33,11 @@ class NET_EXPORT_PRIVATE ProofVerifyDetailsChromium
virtual ProofVerifyDetails* Clone() const OVERRIDE;
CertVerifyResult cert_verify_result;
+
+ // pinning_failure_log contains a message produced by
+ // TransportSecurityState::DomainState::CheckPublicKeyPins in the event of a
+ // pinning failure. It is a (somewhat) human-readable string.
+ std::string pinning_failure_log;
};
// ProofVerifyContextChromium is the implementation-specific information that a
@@ -48,7 +54,8 @@ struct ProofVerifyContextChromium : public ProofVerifyContext {
// capable of handling multiple simultaneous requests.
class NET_EXPORT_PRIVATE ProofVerifierChromium : public ProofVerifier {
public:
- explicit ProofVerifierChromium(CertVerifier* cert_verifier);
+ ProofVerifierChromium(CertVerifier* cert_verifier,
+ TransportSecurityState* transport_security_state);
virtual ~ProofVerifierChromium();
// ProofVerifier interface
@@ -74,6 +81,8 @@ class NET_EXPORT_PRIVATE ProofVerifierChromium : public ProofVerifier {
// Underlying verifier used to verify certificates.
CertVerifier* const cert_verifier_;
+ TransportSecurityState* transport_security_state_;
+
DISALLOW_COPY_AND_ASSIGN(ProofVerifierChromium);
};

Powered by Google App Engine
This is Rietveld 408576698