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

Unified Diff: chrome/browser/net/passive_log_collector.cc

Issue 8965025: Refactoring of job dispatch in HostResolverImpl in preparation for DnsClient. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removed NET_EXPORT_PRIVATE from template. Created 9 years 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: chrome/browser/net/passive_log_collector.cc
diff --git a/chrome/browser/net/passive_log_collector.cc b/chrome/browser/net/passive_log_collector.cc
index 31853872127aad06240b566e25db81defd4a9c9f..dc6e139e76ab92bd6e0d3fcec6a05774849a3b23 100644
--- a/chrome/browser/net/passive_log_collector.cc
+++ b/chrome/browser/net/passive_log_collector.cc
@@ -72,6 +72,8 @@ PassiveLogCollector::PassiveLogCollector()
trackers_[net::NetLog::SOURCE_HOST_RESOLVER_IMPL_REQUEST] =
&dns_request_tracker_;
trackers_[net::NetLog::SOURCE_HOST_RESOLVER_IMPL_JOB] = &dns_job_tracker_;
+ trackers_[net::NetLog::SOURCE_HOST_RESOLVER_IMPL_PROC_JOB] =
+ &dns_proc_job_tracker_;
trackers_[net::NetLog::SOURCE_DISK_CACHE_ENTRY] = &disk_cache_entry_tracker_;
trackers_[net::NetLog::SOURCE_MEMORY_CACHE_ENTRY] = &mem_cache_entry_tracker_;
trackers_[net::NetLog::SOURCE_HTTP_STREAM_JOB] = &http_stream_job_tracker_;
@@ -534,18 +536,20 @@ PassiveLogCollector::SpdySessionTracker::DoAddEntry(
}
//----------------------------------------------------------------------------
-// DNSRequestTracker
+// HostResolverRequestTracker
//----------------------------------------------------------------------------
-const size_t PassiveLogCollector::DNSRequestTracker::kMaxNumSources = 200;
-const size_t PassiveLogCollector::DNSRequestTracker::kMaxGraveyardSize = 20;
+const size_t
+PassiveLogCollector::HostResolverRequestTracker::kMaxNumSources = 200;
+const size_t
+PassiveLogCollector::HostResolverRequestTracker::kMaxGraveyardSize = 20;
-PassiveLogCollector::DNSRequestTracker::DNSRequestTracker()
+PassiveLogCollector::HostResolverRequestTracker::HostResolverRequestTracker()
: SourceTracker(kMaxNumSources, kMaxGraveyardSize, NULL) {
}
PassiveLogCollector::SourceTracker::Action
-PassiveLogCollector::DNSRequestTracker::DoAddEntry(
+PassiveLogCollector::HostResolverRequestTracker::DoAddEntry(
const ChromeNetLog::Entry& entry, SourceInfo* out_info) {
AddEntryToSourceInfo(entry, out_info);
if (entry.type == net::NetLog::TYPE_HOST_RESOLVER_IMPL_REQUEST &&
@@ -556,19 +560,21 @@ PassiveLogCollector::DNSRequestTracker::DoAddEntry(
}
//----------------------------------------------------------------------------
-// DNSJobTracker
+// HostResolverJobTracker
//----------------------------------------------------------------------------
-const size_t PassiveLogCollector::DNSJobTracker::kMaxNumSources = 100;
-const size_t PassiveLogCollector::DNSJobTracker::kMaxGraveyardSize = 15;
+const size_t
+PassiveLogCollector::HostResolverJobTracker::kMaxNumSources = 100;
+const size_t
+PassiveLogCollector::HostResolverJobTracker::kMaxGraveyardSize = 15;
-PassiveLogCollector::DNSJobTracker::DNSJobTracker()
+PassiveLogCollector::HostResolverJobTracker::HostResolverJobTracker()
: SourceTracker(kMaxNumSources, kMaxGraveyardSize, NULL) {
}
PassiveLogCollector::SourceTracker::Action
-PassiveLogCollector::DNSJobTracker::DoAddEntry(const ChromeNetLog::Entry& entry,
- SourceInfo* out_info) {
+PassiveLogCollector::HostResolverJobTracker::DoAddEntry(
+ const ChromeNetLog::Entry& entry, SourceInfo* out_info) {
AddEntryToSourceInfo(entry, out_info);
if (entry.type == net::NetLog::TYPE_HOST_RESOLVER_IMPL_JOB &&
entry.phase == net::NetLog::PHASE_END) {
@@ -578,6 +584,30 @@ PassiveLogCollector::DNSJobTracker::DoAddEntry(const ChromeNetLog::Entry& entry,
}
//----------------------------------------------------------------------------
+// HostResolverProcJobTracker
+//----------------------------------------------------------------------------
+
+const size_t
+PassiveLogCollector::HostResolverProcJobTracker::kMaxNumSources = 100;
+const size_t
+PassiveLogCollector::HostResolverProcJobTracker::kMaxGraveyardSize = 15;
+
+PassiveLogCollector::HostResolverProcJobTracker::HostResolverProcJobTracker()
+ : SourceTracker(kMaxNumSources, kMaxGraveyardSize, NULL) {
+}
+
+PassiveLogCollector::SourceTracker::Action
+PassiveLogCollector::HostResolverProcJobTracker::DoAddEntry(
+ const ChromeNetLog::Entry& entry, SourceInfo* out_info) {
+ AddEntryToSourceInfo(entry, out_info);
+ if (entry.type == net::NetLog::TYPE_HOST_RESOLVER_IMPL_PROC_JOB &&
+ entry.phase == net::NetLog::PHASE_END) {
+ return ACTION_MOVE_TO_GRAVEYARD;
+ }
+ return ACTION_NONE;
+}
+
+//----------------------------------------------------------------------------
// DiskCacheEntryTracker
//----------------------------------------------------------------------------

Powered by Google App Engine
This is Rietveld 408576698