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

Unified Diff: components/certificate_transparency/tree_state_tracker.cc

Issue 2650803004: Wire NetLog into the TreeStateTracker (Closed)
Patch Set: Looking for entry first Created 3 years, 11 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/tree_state_tracker.cc
diff --git a/components/certificate_transparency/tree_state_tracker.cc b/components/certificate_transparency/tree_state_tracker.cc
index 06a74982c96451556bd280114e1841cbdac3ae89..8167e6997a4d14f47f8b05b12fe77e2709c32dc6 100644
--- a/components/certificate_transparency/tree_state_tracker.cc
+++ b/components/certificate_transparency/tree_state_tracker.cc
@@ -33,20 +33,22 @@ 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_TREE_STATE_TRACKER),
+ kMaxConcurrentDnsQueries);
for (const auto& log : ct_logs) {
tree_trackers_[log->key_id()].reset(
- new SingleTreeTracker(log, dns_client_.get()));
+ new SingleTreeTracker(log, dns_client_.get(), net_log));
}
}

Powered by Google App Engine
This is Rietveld 408576698