diff options
| author | mmenke@chromium.org <mmenke@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-24 15:55:51 +0000 |
|---|---|---|
| committer | mmenke@chromium.org <mmenke@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-24 15:55:51 +0000 |
| commit | ee094b89dd380eed93bc0e6ce380d03a0b1b5329 (patch) | |
| tree | d49719e986de616af7b59b95046be1613568a8f7 /jingle | |
| parent | bb872ba65a60c2ade48089f6f0bc538e0cc649ac (diff) | |
| download | chromium_src-ee094b89dd380eed93bc0e6ce380d03a0b1b5329.zip chromium_src-ee094b89dd380eed93bc0e6ce380d03a0b1b5329.tar.gz chromium_src-ee094b89dd380eed93bc0e6ce380d03a0b1b5329.tar.bz2 | |
Adds HostResolveImpl Requests and Jobs to log.
Differences over the reverted version (http://codereview.chromium.org/3080034/show - reverted http://codereview.chromium.org/3137022/show ):
* Jobs never log anything in their destructor, as the HostResolverImpl randomly outlives the NetLog, at least in some unit tests.
* Removed the extra log entries for when a DNS lookup starts/completes. Instead, the job's event is ended in OnLookupComplete().
* Slight modification of the CanceledAsynchronousLookup unit test, as the Job is now closed before posting any events to the Request. For actual lookups, this behavior means the Job's duration more accurately reflects the time the DNS lookup takes itself.
BUG=46844
TEST=Look at the net-internals screen.
Review URL: http://codereview.chromium.org/3119027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57189 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'jingle')
| -rw-r--r-- | jingle/notifier/listener/mediator_thread_impl.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/jingle/notifier/listener/mediator_thread_impl.cc b/jingle/notifier/listener/mediator_thread_impl.cc index c013aa6..63e404e 100644 --- a/jingle/notifier/listener/mediator_thread_impl.cc +++ b/jingle/notifier/listener/mediator_thread_impl.cc @@ -174,7 +174,8 @@ void MediatorThreadImpl::DoLogin( // TODO(akalin): Use an existing HostResolver from somewhere (maybe // the IOThread one). host_resolver_ = - net::CreateSystemHostResolver(net::HostResolver::kDefaultParallelism); + net::CreateSystemHostResolver(net::HostResolver::kDefaultParallelism, + NULL); // Start a new pump for the login. login_.reset(); |