Certificate Transparency: Switch to using a generated CT log list

net/cert/multi_log_ct_verifier.cc

Index: net/cert/multi_log_ct_verifier.cc
diff --git a/net/cert/multi_log_ct_verifier.cc b/net/cert/multi_log_ct_verifier.cc
index 5094da026d03f38bf8cee4c54c7cea5ce2f32580..c0e96c528e1ccd3b2c18734364bad26147021eb8 100644
--- a/net/cert/multi_log_ct_verifier.cc
+++ b/net/cert/multi_log_ct_verifier.cc
@@ -4,6 +4,8 @@
 
 #include "net/cert/multi_log_ct_verifier.h"
 
+#include <vector>

[Ryan Sleevi, 2014/06/19 00:34:08]

Do you actually still need vector? Seems like no.

[Eran Messeri, 2014/06/19 08:28:37]

std::vector is actually used in a different method, I did not include it
properly beforehand.

+
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/metrics/histogram.h"
@@ -66,6 +68,18 @@ void MultiLogCTVerifier::AddLog(scoped_ptr<CTLogVerifier> log_verifier) {
   logs_[log->key_id()] = log;
 }
 
+void MultiLogCTVerifier::AddLogs(
+    ScopedVector<CTLogVerifier> log_verifiers) {
+  for (ScopedVector<CTLogVerifier>::iterator it =
+       log_verifiers.begin(); it != log_verifiers.end(); ++it) {
+    linked_ptr<CTLogVerifier> log(*it);
+    VLOG(1) << "Adding CT log: " << log->description();
+    logs_[log->key_id()] = log;
+  }
+
+  log_verifiers.weak_clear();

[Ryan Sleevi, 2014/06/19 00:34:08]

nit: add comment
// Ownership of the pointers in |log_verifiers| is transferred to |logs_|

[Eran Messeri, 2014/06/19 08:28:37]

Done.

+}
+
 int MultiLogCTVerifier::Verify(
     X509Certificate* cert,
     const std::string& stapled_ocsp_response,