Wire NetLog into the TreeStateTracker

components/certificate_transparency/tree_state_tracker.cc

Index: components/certificate_transparency/tree_state_tracker.cc
diff --git a/components/certificate_transparency/tree_state_tracker.cc b/components/certificate_transparency/tree_state_tracker.cc
index 06a74982c96451556bd280114e1841cbdac3ae89..57260b5ddd6994c9802bb558bca7994eb8350952 100644
--- a/components/certificate_transparency/tree_state_tracker.cc
+++ b/components/certificate_transparency/tree_state_tracker.cc
@@ -33,16 +33,18 @@ const base::Feature kCTLogAuditing = {"CertificateTransparencyLogAuditing",
                                       base::FEATURE_DISABLED_BY_DEFAULT};
 
 TreeStateTracker::TreeStateTracker(
-    std::vector<scoped_refptr<const CTLogVerifier>> ct_logs) {
+    std::vector<scoped_refptr<const CTLogVerifier>> ct_logs,
+    net::NetLog* net_log) {
   if (!base::FeatureList::IsEnabled(kCTLogAuditing))
     return;
 
-  // TODO(eranm): Hook up a real NetLog.
-  net::NetLogWithSource net_log;
   std::unique_ptr<net::DnsClient> dns_client =
-      net::DnsClient::CreateClient(net_log.net_log());
-  dns_client_ = base::MakeUnique<LogDnsClient>(std::move(dns_client), net_log,
-                                               kMaxConcurrentDnsQueries);
+      net::DnsClient::CreateClient(net_log);
+  dns_client_ = base::MakeUnique<LogDnsClient>(
+      std::move(dns_client),
+      net::NetLogWithSource::Make(
+          net_log, net::NetLogSourceType::CT_LOG_AUDITOR),

[eroman, 2017/01/25 21:23:08]

An alternate name could be CT_TREE_STATE_TRACKER, up to you.

[Eran Messeri, 2017/01/26 15:08:40]

Done, as it'd be more consistent with the name of the class (but TBH I think the
class is not named particularly well. How about renaming it to CTLogAuditor ?)

[eroman, 2017/01/26 18:20:39]

sgtm

+      kMaxConcurrentDnsQueries);
 
   for (const auto& log : ct_logs) {
     tree_trackers_[log->key_id()].reset(