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

Unified Diff: components/certificate_transparency/single_tree_tracker_unittest.cc

Issue 2017563002: Add Certificate Transparency logs auditing (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Switching to TimestampedLeaf Created 4 years, 6 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: components/certificate_transparency/single_tree_tracker_unittest.cc
diff --git a/components/certificate_transparency/single_tree_tracker_unittest.cc b/components/certificate_transparency/single_tree_tracker_unittest.cc
index cbdfe6175b8702c1532a6830790c7d2deba04836..d5065c94a7c6d85e3ddc4d9e5e365153baf83f55 100644
--- a/components/certificate_transparency/single_tree_tracker_unittest.cc
+++ b/components/certificate_transparency/single_tree_tracker_unittest.cc
@@ -63,6 +63,13 @@ class SingleTreeTrackerTest : public ::testing::Test {
der_test_cert.length());
ASSERT_TRUE(chain_.get());
net::ct::GetX509CertSCT(&cert_sct_);
+ // Because the code will ultimately try to build a MerkleTreeLeaf from
+ // the |chain_| and |cert_sct_|, if the default type, which is an embedded
+ // SCT, is used, leaf construction will fail because GetMerkleTreeLeaf will
+ // expect an intermediate. So change SCT origin to something that relies
+ // solely on |chain_|.
+ cert_sct_->origin =
+ net::ct::SignedCertificateTimestamp::SCT_FROM_OCSP_RESPONSE;
}
protected:

Powered by Google App Engine
This is Rietveld 408576698