Chromium Code Reviews| Index: net/base/cert_verify_proc_unittest.cc |
| =================================================================== |
| --- net/base/cert_verify_proc_unittest.cc (revision 152152) |
| +++ net/base/cert_verify_proc_unittest.cc (working copy) |
| @@ -120,7 +120,7 @@ |
| ASSERT_NE(static_cast<X509Certificate*>(NULL), paypal_null_cert); |
| - const SHA1Fingerprint& fingerprint = |
| + const SHA1HashValue& fingerprint = |
| paypal_null_cert->fingerprint(); |
| for (size_t i = 0; i < 20; ++i) |
| EXPECT_EQ(paypal_null_fingerprint[i], fingerprint.data[i]); |
| @@ -397,10 +397,11 @@ |
| std::string spki_sha1 = base::SHA1HashString(spki.as_string()); |
| - std::vector<SHA1Fingerprint> public_keys; |
| - SHA1Fingerprint fingerprint; |
| - ASSERT_EQ(sizeof(fingerprint.data), spki_sha1.size()); |
| - memcpy(fingerprint.data, spki_sha1.data(), spki_sha1.size()); |
| + HashValueVector public_keys; |
| + HashValue fingerprint; |
|
Ryan Sleevi
2012/08/23 22:48:20
nit:
HashValue fingerprint(HASH_VALUE_SHA1);
palmer
2012/08/29 00:01:24
Done, plus s/fingerprint/hash/ per hshi1's comment
|
| + fingerprint.tag = HASH_VALUE_SHA1; |
| + ASSERT_EQ(fingerprint.size(), spki_sha1.size()); |
| + memcpy(fingerprint.data(), spki_sha1.data(), spki_sha1.size()); |
| public_keys.push_back(fingerprint); |
| EXPECT_TRUE(CertVerifyProc::IsPublicKeyBlacklisted(public_keys)) << |
| @@ -454,9 +455,18 @@ |
| EXPECT_EQ(OK, error); |
| EXPECT_EQ(0U, verify_result.cert_status); |
| ASSERT_LE(3u, verify_result.public_key_hashes.size()); |
| - for (unsigned i = 0; i < 3; i++) { |
| + |
| + HashValueVector sha1_hashes; |
| + for (unsigned i = 0; i < verify_result.public_key_hashes.size(); ++i) { |
| + if (verify_result.public_key_hashes[i].tag != HASH_VALUE_SHA1) |
| + continue; |
| + sha1_hashes.push_back(verify_result.public_key_hashes[i]); |
| + } |
| + ASSERT_LE(3u, sha1_hashes.size()); |
| + |
| + for (unsigned i = 0; i < 3; ++i) { |
| EXPECT_EQ(HexEncode(kCertSESPKIs[i], base::kSHA1Length), |
| - HexEncode(verify_result.public_key_hashes[i].data, base::kSHA1Length)); |
| + HexEncode(sha1_hashes[i].data(), base::kSHA1Length)); |
| } |
| } |