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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/cert/cert_verify_result.h

Issue 2672083003: Re-enable CertVerifyProcInternalTest.PublicKeyHashes. (Closed)
Patch Set: switch to EXPECT_THAT() Created 3 years, 10 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
« no previous file with comments | « net/cert/cert_verify_proc_unittest.cc ('k') | net/test/test_certificate_data.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/cert/cert_verify_result.h
diff --git a/net/cert/cert_verify_result.h b/net/cert/cert_verify_result.h
index 176c4ca50ed4ca379429c2eb38ee641bf1a8faa7..5077e2fd349e47dbf5b481d9af7ebd5ddc965d9f 100644
--- a/net/cert/cert_verify_result.h
+++ b/net/cert/cert_verify_result.h
@@ -61,8 +61,11 @@ class NET_EXPORT CertVerifyResult {
bool has_sha1_leaf;
// If the certificate was successfully verified then this contains the
- // hashes, in several hash algorithms, of the SubjectPublicKeyInfos of the
- // chain.
+ // hashes for all of the SubjectPublicKeyInfos of the chain (target,
+ // intermediates, and trust anchor)
+ //
+ // The ordering of the hashes in this vector is unspecified. Both the SHA1
+ // and SHA256 hash will be present for each certificate.
HashValueVector public_key_hashes;
// is_issued_by_known_root is true if we recognise the root CA as a standard
« no previous file with comments | « net/cert/cert_verify_proc_unittest.cc ('k') | net/test/test_certificate_data.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698