summaryrefslogtreecommitdiffstats
path: root/net/base/host_resolver_impl.cc
Commit message (Collapse)AuthorAgeFilesLines
* Remove obsolete base/lock.h and fix up callers to use the new header file andbrettw@chromium.org2011-01-211-7/+7
| | | | | | | | | | | the base namespace. Fix several files including lock.h unnecessarily. BUG=none TEST=none Original review=http://codereview.chromium.org/6142009/ Patch by leviw@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72106 0039d316-1c4b-4281-b951-d872f2087c98
* Even more reordering the methods in headers and implementation in net/.erg@google.com2011-01-201-46/+46
| | | | | | | | | BUG=68682 TEST=compiles Review URL: http://codereview.chromium.org/6314010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71880 0039d316-1c4b-4281-b951-d872f2087c98
* Move some misc thread-related stuff from base to base/thread and into the basebrettw@chromium.org2010-12-301-3/+3
| | | | | | | | | | namespace. This does not move the "hard" thread stuff (thread.h). TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6079009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70315 0039d316-1c4b-4281-b951-d872f2087c98
* Even more virtual method deinlining.erg@google.com2010-12-131-0/+4
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5741001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69041 0039d316-1c4b-4281-b951-d872f2087c98
* Convert implicit scoped_refptr constructor calls to explicit ones, part 2thakis@chromium.org2010-11-021-6/+10
| | | | | | | | | | | | This CL was created automatically by this clang rewriter: http://codereview.appspot.com/2826041 I then did quite a bit of manual editing to fix style issues. BUG=28083 TEST=None Review URL: http://codereview.chromium.org/4291001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64798 0039d316-1c4b-4281-b951-d872f2087c98
* Convert implicit scoped_refptr constructor calls to explicit ones, part 1thakis@chromium.org2010-11-011-3/+3
| | | | | | | | | | | This CL was created automatically by this clang rewriter: http://codereview.appspot.com/2776043/ . I manually fixed a few rough spots of the rewriter output (doh1-3) and fixed all presubmit errors. BUG=28083 TEST=None Review URL: http://codereview.chromium.org/4192012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64573 0039d316-1c4b-4281-b951-d872f2087c98
* Restrict total parallel DNS resolutionsjar@chromium.org2010-10-291-3/+5
| | | | | | | | | | | | | | | | | | | A/B tests suggest that large numbers of parallel resolutions may cause the DNS failure rate (re: ratio of "host not found" to "found") to rise. To stay safely away from that threshold, this change lowers the maximum parallel resolutions to 8, and restricts the speculative resolutions to 3. We are also running A/B tests which will look at the impact of modulating either of these values. BUG=3041 r=eroman Review URL: http://codereview.chromium.org/4111004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64369 0039d316-1c4b-4281-b951-d872f2087c98
* Initial support for built-in DNS resolver/cache.agayev@chromium.org2010-10-281-1/+2
| | | | | | | | | BUG=60149 TEST=None Review URL: http://codereview.chromium.org/4216001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64271 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 64202 - Initial support for built-in DNS resolver/cache.yusukes@chromium.org2010-10-281-2/+1
| | | | | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/3693001 TBR=agayev@chromium.org Review URL: http://codereview.chromium.org/4208002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64204 0039d316-1c4b-4281-b951-d872f2087c98
* Initial support for built-in DNS resolver/cache.agayev@chromium.org2010-10-281-1/+2
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/3693001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64202 0039d316-1c4b-4281-b951-d872f2087c98
* Move debug-related stuff from base to the base/debug directory and use thebrettw@chromium.org2010-10-261-2/+3
| | | | | | | | | | | | | | | | base::debug namespace. This splits apart debug_util into base/debugger and base/stack_trace There are still two functions in debug_util that I'm not sure what to do with. Since this uses the base::debug namespace, I removed the functions in debugger.h from the static class and just made them free functions in the namespace. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3945002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63859 0039d316-1c4b-4281-b951-d872f2087c98
* A/B test of parelellism limits on DNS resolutionsjar@chromium.org2010-10-201-2/+14
| | | | | | | | | | | | | | | | Current default is a max of 50 parallel resolutions. This test tries to set limits at 6, 8, 10, 14, and 20. Stats gathered include the number of successful resolutions, and well as the number of failed resolutions. We're looking to see if the ratio changes as a function of the parallelism limit. BUG=3041 r=eroman Review URL: http://codereview.chromium.org/4010002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63297 0039d316-1c4b-4281-b951-d872f2087c98
* Fix clang warning about local types being used as template parameters.thakis@chromium.org2010-10-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | clang would complain /Users/thakis/src/chrome-git/src/net/base/host_resolver_impl.cc:557:5: error: template argument uses local type 'net::HostResolverImpl::Job::Category' [-Wlocal-type-template-args] DCHECK_LT(category, RESOLVE_MAX); // Be sure it was set. ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ In file included from /Users/thakis/src/chrome-git/src/net/base/host_resolver_impl.cc:5: In file included from ../net/base/host_resolver_impl.h:18: In file included from ../net/base/network_change_notifier.h:10: In file included from ../base/observer_list_threadsafe.h:14: ../base/logging.h:662:31: note: instantiated from: #define DCHECK_LT(val1, val2) DCHECK_OP(LT, < , val1, val2) ^ ../base/logging.h:635:9: note: instantiated from: logging::Check##name##Impl((val1), (val2), \ ^~~~~~~ This seems to be a new warning. Since it only affects this file, we might as well fix the problem instead of disabling the warning. BUG=None TEST=clang/mac waterfall stays green after clobber build. Review URL: http://codereview.chromium.org/3816009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62934 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a bad merge.eroman@chromium.org2010-10-161-1/+1
| | | | | | | | | | The check-in r62746 clobbered the changes I made in r62689 to host_resolver_impl.cc. I believe that was an accident since r62746 moved code within the file. TBR=jar Review URL: http://codereview.chromium.org/3828008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62879 0039d316-1c4b-4281-b951-d872f2087c98
* Convert LOG(INFO) to VLOG(1) - net/.pkasting@chromium.org2010-10-161-4/+3
| | | | | | | | | | Also converts COOKIE_DLOG (since VLOG can be toggled on a granular basis), removes some unneeded "endl"s and {}s, aligns "<<"s per style guide, and changes KeygenHandler::GenKeyAndSignChallenge() to avoid using "goto". BUG=none TEST=none Review URL: http://codereview.chromium.org/3846001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62831 0039d316-1c4b-4281-b951-d872f2087c98
* Add performance histograms to resolver, asong with A/B testjar@chromium.org2010-10-151-13/+61
| | | | | | | | | | | | | | | | | | | | | | I'm doing tests to see what the impact of changing the speculative resolution system parameters is on resolution time. By default, the speculative system limits itself to no more than 8 pending resolutions at any time, and this test looks at 6, 4, and 2 as limits. It also modulates the speculative congestion avoidance system, which purges the speculative queue when there is too much resolution latency, and tries various durations of time. (the latter was less important... but was already part of this existing A/B test.) I also pushed the test rates up to 10% for each case, so that we'd get data from the canary and/or dev builds. r=eroman Review URL: http://codereview.chromium.org/3762006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62746 0039d316-1c4b-4281-b951-d872f2087c98
* Return a distinct error from ERR_NAME_NOT_RESOLVED when the call to ↵eroman@chromium.org2010-10-151-1/+1
| | | | | | | | | | | | | | | getaddrinfo() has unexpectedly failed. For example when the system returns WSA_NOT_ENOUGH_MEMORY, we will now show ERR_NAME_RESOLUTION_FAILED rather than ERR_NAME_NOT_RESOLVED. This allows propagating the error to the user, rather than simply showing them say the linkdoctor page as if the domain they entered was incorrect. The data shows that a small number of users are getting these sorts of unexpected failures, and I want to get more visibility into it. By having a separate error code we can track how often this actually floats up onto an error page (as opposed to just errors on non-essential background requests). On Linux we also see EAI_AGAIN with some frequency. My theory is that these occur when you DNS resolve immediately after the network change notification was broadcast (since I can repro this 100% on my laptop). If this is in fact the major cause, then once I land this new error we should NOT see an influx of ERR_NAME_RESOLUTION_FAILED in Net.ErrorCodesForMainFrame. Review URL: http://codereview.chromium.org/3781005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62705 0039d316-1c4b-4281-b951-d872f2087c98
* Rename OSErrorForGetAddrinfo to include a suffix for the particular platform.eroman@chromium.org2010-10-151-1/+15
| | | | | | | This makes it easier to display the symbolic names of each error code on the server side, since we specify a mapping per histogram name. Review URL: http://codereview.chromium.org/3771005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62689 0039d316-1c4b-4281-b951-d872f2087c98
* Move Stats, histograms, and field trial into a metrics subdirectory of base andbrettw@chromium.org2010-10-141-1/+1
| | | | | | | | | put them in the base namespace. TEST=it compiles BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62510 0039d316-1c4b-4281-b951-d872f2087c98
* Return ERR_INTERNET_DISCONNECTED in place of ERR_NAME_NOT_RESOLVED and ↵eroman@chromium.org2010-10-091-0/+6
| | | | | | | | | | | | ERR_ADDRESS_UNREACHABLE, when the user is in offline mode. This initial changelist includes the implementation just for Windows. BUG=53473 TEST=unplug network cable, try to connect to www.google.com --> should get the error ERR_INTERNET_DISCONNECTED. Next, try connecting directly to an IP address --> should get the error ERR_INTERNET_DISCONNECTED. Review URL: http://codereview.chromium.org/3634002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62050 0039d316-1c4b-4281-b951-d872f2087c98
* Shows the default_address_family on both the DNS tab and in log dumps,mmenke@chromium.org2010-10-071-0/+4
| | | | | | | | | | | | | | | and adds an extra note emphasizing when IPv6 has been disabled to both pages. Also adds a button to enable IPv6 when it's been disabled, and maps address family values on the DNS tab to their symbolic names. BUG=55007,55005 TEST=manual Review URL: http://codereview.chromium.org/3387007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61795 0039d316-1c4b-4281-b951-d872f2087c98
* HostResolver should flush the JobPools on a network change.willchan@chromium.org2010-09-211-3/+9
| | | | | | | | | BUG=56291 TEST=HostResolverImplTest.ObeyPoolConstraintsAfterIPAddressChange Review URL: http://codereview.chromium.org/3431017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60127 0039d316-1c4b-4281-b951-d872f2087c98
* Change HostResolver::RequestInfo to take a HostPortPair rather than naked ↵eroman@chromium.org2010-09-151-2/+1
| | | | | | | | host string + port. Review URL: http://codereview.chromium.org/3420001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59577 0039d316-1c4b-4281-b951-d872f2087c98
* Fix remaining localhost resolution issues.vandebo@chromium.org2010-09-031-17/+29
| | | | | | | | | | | | | If host resolution is restricted to a single address family and the results of a resolution are all localhost of that type, try again without the address family restriction. Also make --disable-ipv6 apply to resolution of ip literals. BUG=42058, 49024, 32522 TEST=Manual testing in various network configurations. See https://spreadsheets.google.com/ccc?key=0AhSnKEusL6UgdFZIRmxTUnYtZDhjX3lKclBqMHo4YUE&hl=en&authkey=CLKDxMYB Review URL: http://codereview.chromium.org/3231005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58534 0039d316-1c4b-4281-b951-d872f2087c98
* Fix invalid read in HostResolverImpl::OnIPAddressChanged.willchan@chromium.org2010-09-011-1/+2
| | | | | | | | | BUG=54090 TEST=HostResolverImplTest.OnlyAbortExistingRequestsOnIPAddressChange Review URL: http://codereview.chromium.org/3348001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58209 0039d316-1c4b-4281-b951-d872f2087c98
* Relands r58007.willchan@chromium.org2010-09-011-6/+34
| | | | | | | | | | Fixes the problem with the flaky unittest that happened due to message loop posting race conditions. BUG=53386 TEST=HostResolverImplTest.AbortOnIPAddressChanged,HostResolverImplTest.OnlyAbortExistingRequestsOnIPAddressChange Review URL: http://codereview.chromium.org/3231013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58171 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r58019 and r58007 due to flaky.willchan@chromium.org2010-08-311-34/+6
| | | | | | | | | Blargh, there are race conditions in UI->IO MessageLoop passing. Need to fix them. BUG=53965,53386 Review URL: http://codereview.chromium.org/3215012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58023 0039d316-1c4b-4281-b951-d872f2087c98
* Abort host resolution requests with ERR_ABORTED on ip address change.willchan@chromium.org2010-08-311-6/+34
| | | | | | | | BUG=53386 Review URL: http://codereview.chromium.org/3277002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58007 0039d316-1c4b-4281-b951-d872f2087c98
* Adds HostResolveImpl Requests and Jobs to log.mmenke@chromium.org2010-08-241-54/+182
| | | | | | | | | | | | | | | 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
* Don't resolve IP literals.vandebo@chromium.org2010-08-181-2/+17
| | | | | | | | | | | | | For each resolution request this checks to see if this 'host' is a literal ip address. If so, it synthesises a struct addrinfo and returns it without adding it to the cache. BUG=39830 TEST=unit tests, new and old Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=56384 Review URL: http://codereview.chromium.org/3023048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56615 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 56539 - Added HostResolveImpl Requests and Jobs to log.mmenke@chromium.org2010-08-181-183/+51
| | | | | | | | | | | | | | | | | | ConnectJobs point to the requests, Requests point back to ConnectJobs and to the DNS lookup they were attached to, if any. Also CONNECT_JOBs are now identified by their host/port on the Requests list. BUG= 46844 TEST= Look at the net-internals screen. Review URL: http://codereview.chromium.org/3080034 TBR=eroman@chromium.org Review URL: http://codereview.chromium.org/3137022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56556 0039d316-1c4b-4281-b951-d872f2087c98
* Added HostResolveImpl Requests and Jobs to log.mmenke@chromium.org2010-08-181-51/+183
| | | | | | | | | | | | | | | ConnectJobs point to the requests, Requests point back to ConnectJobs and to the DNS lookup they were attached to, if any. Also CONNECT_JOBs are now identified by their host/port on the Requests list. BUG= 46844 TEST= Look at the net-internals screen. Review URL: http://codereview.chromium.org/3080034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56539 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 56384 - Don't resolve IP literals.vandebo@chromium.org2010-08-171-17/+2
| | | | | | | | | | | | | | For each resolution request this checks to see if this 'host' is a literal ip address. If so, it synthesises a struct addrinfo and returns it without adding it to the cache. BUG=39830 TEST=unit tests, new and old Review URL: http://codereview.chromium.org/3023048 TBR=vandebo@chromium.org Review URL: http://codereview.chromium.org/3115014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56385 0039d316-1c4b-4281-b951-d872f2087c98
* Don't resolve IP literals.vandebo@chromium.org2010-08-171-2/+17
| | | | | | | | | | | For each resolution request this checks to see if this 'host' is a literal ip address. If so, it synthesises a struct addrinfo and returns it without adding it to the cache. BUG=39830 TEST=unit tests, new and old Review URL: http://codereview.chromium.org/3023048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56384 0039d316-1c4b-4281-b951-d872f2087c98
* Convert src/net to use std::string/char* for DictionaryValue keys.viettrungluu@chromium.org2010-07-311-5/+6
| | | | | | | | | | | Hopefully I got them all. BUG=23581 TEST=net_unittests Review URL: http://codereview.chromium.org/3013048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54433 0039d316-1c4b-4281-b951-d872f2087c98
* Implement prefetching in chromegavinp@google.com2010-07-311-1/+2
| | | | | | | | | | | | | | | | | | With this CL (see also issue 2910009), chrome will support basic prefetching. You can optionally deactivate prefetching with the command line argument --disable-prefetch. A new RequestPriority was created as well, IDLE, which is lower than LOWEST. Unfortunately, SPDY has only two bits for priority, so as a temporary measure (pending SPDY v3 which will have three), we have a mapping in SPDY that folds net::LOWEST and net::IDLE together. BUG=13505 TEST=http://gemal.dk/browserspy/prefetch.php Review URL: http://codereview.chromium.org/3050016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54421 0039d316-1c4b-4281-b951-d872f2087c98
* Avoid printing log errors during shutdown of HostResolverImpl when there are ↵eroman@chromium.org2010-07-271-1/+2
| | | | | | | | | still requests outstanding. BUG=49692 Review URL: http://codereview.chromium.org/3023011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53837 0039d316-1c4b-4281-b951-d872f2087c98
* [Linux] Enable connecting to localhost when offline.vandebo@chromium.org2010-07-231-2/+14
| | | | | | | | | | | | | Add a utility function to determine if only loopback address are configured. Add a mechanism to add supplemental HostResolverFlags in the HostResolver. Change the resolver on Linux to not use AI_ADDRCONFIG if only loopback addresses are configured. BUG=41408 TEST=localhost works when offline Review URL: http://codereview.chromium.org/3036011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53487 0039d316-1c4b-4281-b951-d872f2087c98
* Add a command line flag to change the default number of parallel DNS ↵eroman@chromium.org2010-07-171-3/+7
| | | | | | | | | | | | | requests issued by chrome. --host-resolver-parallelism=XXX This is a useful debugging tool, since for some users too many parallel DNS requests results in spurious DNS failures from their resolver. BUG=44489 Review URL: http://codereview.chromium.org/3019007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52802 0039d316-1c4b-4281-b951-d872f2087c98
* Attempt four at landing the "histogram.h removed from message_loop.h" patch.erg@chromium.org2010-07-151-0/+1
| | | | | | | | | | | | | | Previously committed as r52349 and r52336. Related commits: r52367, r52364 and r52343. Rerunning trybots due to previous trybot breakage. TEST=none BUG=none Review URL: http://codereview.chromium.org/2965015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52496 0039d316-1c4b-4281-b951-d872f2087c98
* Revert everything related to the relanding.erg@chromium.org2010-07-141-1/+0
| | | | | | Review URL: http://codereview.chromium.org/2982009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52367 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r52336.erg@chromium.org2010-07-141-0/+1
| | | | | | | | | | | | | | (I accidentally committed a previous version of the patch (the fixed version was in codereview since I had switched to my mac to address this exact compile failure.) BUG=none TEST=none TBR=evan Review URL: http://codereview.chromium.org/2937010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52349 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 52336 - More header cleanup:erg@chromium.org2010-07-141-1/+0
| | | | | | | | | | | | | | | | - Some cleanup to url_request.h - Remove task.h from headers that don't need it. - Remove histogram.h from message_loop.h BUG=none TEST=none Review URL: http://codereview.chromium.org/2986002 TBR=erg@chromium.org Review URL: http://codereview.chromium.org/2969006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52343 0039d316-1c4b-4281-b951-d872f2087c98
* More header cleanup:erg@chromium.org2010-07-141-0/+1
| | | | | | | | | | | | | - Some cleanup to url_request.h - Remove task.h from headers that don't need it. - Remove histogram.h from message_loop.h BUG=none TEST=none Review URL: http://codereview.chromium.org/2986002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52336 0039d316-1c4b-4281-b951-d872f2087c98
* Massively simplify the NetworkChangeNotifier infrastructure:pkasting@chromium.org2010-06-251-11/+5
| | | | | | | | | | | | | | * Use a process-wide object (singleton pattern) * Create/destroy this object on the main thread, make it outlive all consumers * Make observer-related functions threadsafe As a result, the notifier can now be used by any thread (eliminating things like NetworkChangeObserverProxy and NetworkChangeNotifierProxy, and expanding its usefulness); its creation and inner workings are much simplified (eliminating implementation-specific classes); and it is simpler to access (eliminating things like NetworkChangeNotifierThread and a LOT of passing pointers around). BUG=none TEST=Unittests; network changes still trigger notifications Review URL: http://codereview.chromium.org/2802015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50895 0039d316-1c4b-4281-b951-d872f2087c98
* Disable caching of failed DNS lookups.eroman@chromium.org2010-06-171-1/+1
| | | | | | | | | | | | This is causing problems on network changes, since we don't have a reliable signal for telling when DNS is back up (so we aren't able to flush the cache properly). This will regress bug 1672, filed 46821 to address that. BUG=46821 Review URL: http://codereview.chromium.org/2873003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50141 0039d316-1c4b-4281-b951-d872f2087c98
* Make HostResolver NonThreadSafe and not thread safe refcounted.willchan@chromium.org2010-06-031-0/+9
| | | | | | | | | | Required making SyncHostResolverBridge not use RefCountedThreadSafe. I refactored the innards to have a thread safe refcounted Core implementation. BUG=45298 Review URL: http://codereview.chromium.org/2122015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48867 0039d316-1c4b-4281-b951-d872f2087c98
* Log the OS error that getaddrinfo() failed with.eroman@chromium.org2010-05-161-19/+139
| | | | | | | | | - Display it on the about:net-internals page - Collect it in the histogram Net.OSErrorsForGetAddrinfo. Review URL: http://codereview.chromium.org/2134004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47386 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the HOST_RESOLVER_IMPL_OBSERVER_* log events. I have never found ↵eroman@chromium.org2010-05-111-12/+0
| | | | | | | | these to be useful, and they just add noise to the log. Review URL: http://codereview.chromium.org/2033008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46885 0039d316-1c4b-4281-b951-d872f2087c98
* Replace about:net-internals with the javascript-based frontend.eroman@chromium.org2010-05-101-168/+2
| | | | | | | | | | (DNS request tracing is the only feature lost in this transition; it needs to be added back under the new framework). BUG=37421 Review URL: http://codereview.chromium.org/2008007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46868 0039d316-1c4b-4281-b951-d872f2087c98