Add Certificate Transparency logs auditing

components/certificate_transparency/single_tree_tracker_unittest.cc

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_|.

[Ryan Sleevi, 2016/06/30 22:48:19]

LAYERING: This comment strikes me as over-explaining something future /
unrelated.

Are you sure this comment belongs?

[Eran Messeri, 2016/07/01 13:24:01]

I've simply removed the comment. Regardless of the explanation there it's the
right thing to set.

+    cert_sct_->origin =
+        net::ct::SignedCertificateTimestamp::SCT_FROM_OCSP_RESPONSE;
   }
 
  protected: