summaryrefslogtreecommitdiffstats
path: root/net
Commit message (Collapse)AuthorAgeFilesLines
* net: comment/TODO cleanup and clarification, and don't raise ↵rsleevi@chromium.org2010-10-202-18/+25
| | | | | | | | | | | | ERR_SSL_PROTOCOL_ERROR for user_cancelled TLS alerts on Mac. TBR=wtc BUG=none TEST=none Review URL: http://codereview.chromium.org/3938002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63199 0039d316-1c4b-4281-b951-d872f2087c98
* Fix loading of a temporary root certificate for OpenSSLrsleevi@chromium.org2010-10-201-9/+26
| | | | | | | | | BUG=None TEST=Build with use_openssl=1, HTTPSRequestTest.HTTPSExpiredTest Review URL: http://codereview.chromium.org/3678001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63189 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Even more ctor/virtual deinlining.erg@google.com2010-10-196-14/+64
| | | | | | | | | | | (Only 424k off Linux debug .a files). BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3859003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63059 0039d316-1c4b-4281-b951-d872f2087c98
* Revert revert 62918 - net: clean up SSLHostInfo construction."agl@chromium.org2010-10-1942-91/+227
| | | | | | | Was reverted in r62922 due to a change landing between the try run and the commit. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63046 0039d316-1c4b-4281-b951-d872f2087c98
* Moving the disable test to correct file.oshima@chromium.org2010-10-191-0/+3
| | | | | | | | | BUG=59642 TEST=win32 tsan bot goes green. Review URL: http://codereview.chromium.org/3808011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62995 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Monster ctor patch after changing heuristics in clang plugin.erg@google.com2010-10-1843-106/+275
| | | | | | | | | | | (Only 916k this time off Debug Linux .a files) BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3814013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62967 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Initialize data members in constructor.kmadhusu@chromium.org2010-10-184-2/+8
| | | | | | | | | | CID=1634, 11587, 5007, 13234, 13398, 9295, 8294, 8285 BUG=none TEST=none Review URL: http://codereview.chromium.org/3817007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62945 0039d316-1c4b-4281-b951-d872f2087c98
* Don't rely on "access_time < Time::Now()" to indicate "all cookies".rdsmith@google.com2010-10-183-11/+6
| | | | | | | | | | | | | | | | The interface to CookieMonster::GarbageCollectDeleteList() was signalling "delete all cookies in list" by specifying "delete all cookies in list with last access time before Now()". This was failing to work properly when the system clock wasn't updated frequently enough (== flaky test) and is also vulnerable to changes in systems clocks. That semantic is now explicitly signalled by a null time. BUG=58197 TEST=net_unittests CookieMonsterTest.* on Win/Linux and CookieMonsterTest.TestHostGarbageCollection (flaky test) repeatedly on Windows. Review URL: http://codereview.chromium.org/3780004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62943 0039d316-1c4b-4281-b951-d872f2087c98
* Fix poorly performancing implementation of MockSimplePersistentCookieStorerdsmith@google.com2010-10-182-28/+18
| | | | | | | | | BUG=57686 TEST=net_unittests CookieMonsterTest.* on Linux, running original valgrind test and confirming that it completed. Review URL: http://codereview.chromium.org/3752004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62942 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
* Virtual destructor cleanup: RRResolverHandle::Run does not need to be virtual.cbentzel@chromium.org2010-10-181-2/+2
| | | | | | | | BUG=45135 TEST=Build works. Review URL: http://codereview.chromium.org/3747008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62927 0039d316-1c4b-4281-b951-d872f2087c98
* non-virtual-dtor cleanup: Make DNSSecProvider destructor private.cbentzel@chromium.org2010-10-181-0/+3
| | | | | | | | | | Interface classes are an example where the -Wnon-virtual-dtor warning can be a bit annoying, but this still seems reasonable and makes it clear that users should not delete through the interface. BUG=45135 TEST=Builds Review URL: http://codereview.chromium.org/3781010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62926 0039d316-1c4b-4281-b951-d872f2087c98
* Caches DER data in X509 certificate for OpenSSL.bulach@chromium.org2010-10-181-26/+94
| | | | | | | | | BUG=none TEST=net_unittests --gtest_filter=*X509* Review URL: http://codereview.chromium.org/3785001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62925 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 62918 - net: clean up SSLHostInfo construction.agl@chromium.org2010-10-1841-225/+90
| | | | | | | | | | | | | | | | Create an SSLHostInfoFactory interface and plumb it from the HttpCache to the SSLConnectJob. Also, move the SSLHostInfo reference from the ssl_config to being passed to the SSLClientSocket. BUG=none TEST=net_unittests Review URL: http://codereview.chromium.org/3747003 TBR=agl@chromium.org Review URL: http://codereview.chromium.org/3846005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62922 0039d316-1c4b-4281-b951-d872f2087c98
* net: clean up SSLHostInfo construction.agl@chromium.org2010-10-1841-90/+225
| | | | | | | | | | | | | Create an SSLHostInfoFactory interface and plumb it from the HttpCache to the SSLConnectJob. Also, move the SSLHostInfo reference from the ssl_config to being passed to the SSLClientSocket. BUG=none TEST=net_unittests Review URL: http://codereview.chromium.org/3747003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62918 0039d316-1c4b-4281-b951-d872f2087c98
* Move pe_image and registry from base to base/win and use the namespace. It ↵brettw@chromium.org2010-10-173-8/+9
| | | | | | | | | | | | removes windows_message_list which isn't used. This keeps a stub for registry in the old location until we can update that. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3836005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62888 0039d316-1c4b-4281-b951-d872f2087c98
* Move scoped_cftyperef from base to base/mac, use the new namespace, and name itbrettw@chromium.org2010-10-1711-54/+61
| | | | | | | | | | properly (scoped_cftyperef -> ScopedCFTypeRef). TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3855001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62887 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
* Cleanup: Move the ProxyScriptFetcher registry from being a global in net, to ↵eroman@chromium.org2010-10-167-214/+188
| | | | | | | | | | | | living in IOThread. I had to make some other changes to make this fit well: moved ProxyScriptFetcherImpl to its own set of files, and added a IOThread::Get() accessor to avoid plumbing through several layers in connection_tester. I find the registry living in IOThread is preferable, as globals in net/ limit the ability to run on safely on multiple threads, and also leads to confusion on what needs to be called at shutdown. Review URL: http://codereview.chromium.org/3823001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62876 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 61897 - Do not use Alternate-Protocol for establishing new SpdySessions.mbelshe@chromium.org2010-10-165-78/+5
| | | | | | | | | | | | | | | | | | | | | Only use Alternate-Protocol for latching onto existing SpdySessions. This is a temporary change until SSL improvements are made, so we can begin sending http over spdy when available. Note that we have to do proxy resolution one time to figure out how to check for the appropriate existing SpdySession. If no SpdySession exists, we fallback again and have to redo proxy resolution (since the alternate protocol of npn-spdy requires rewriting the URI from http:// to https://). So we may end up doing two proxy resolutions in the fallback case. I had to disable all the tests that test establishing a new SpdySession for http:// URIs, since we don't do that for now. BUG=none TEST=none Review URL: http://codereview.chromium.org/3573013 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/3751009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62873 0039d316-1c4b-4281-b951-d872f2087c98
* Add an off switch for SPDY:mbelshe@chromium.org2010-10-167-25/+109
| | | | | | | | | | | --use-spdy=off BUG=none TEST=SpdyNetworkTransactionTest.SpdyOnOffToggle Review URL: http://codereview.chromium.org/3807008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62870 0039d316-1c4b-4281-b951-d872f2087c98
* Move xdg_util into base/nix. I originally wanted to use base/linux, but youbrettw@chromium.org2010-10-161-9/+9
| | | | | | | | | | | | | | | | can't have a namespace called "linux" because it's ifdefed by GCC to be a number. Same for "unix". "x11" was discarded because some future stuff in here will not be related to x11. This also renames base_paths_posix to base_paths_linux since this does not apply to mac, and normally posix applies to mac (this confused me when I had to edit it for the XDG stuff). TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3778007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62848 0039d316-1c4b-4281-b951-d872f2087c98
* Move the windows-specific scoped_* stuff from base to base/win and in the ↵brettw@chromium.org2010-10-161-3/+4
| | | | | | | | | | | | | | | | base::win namespace. This keeps old headers that forward to the new versions and have using declarations that allow the existing code to compile. I fixed all the callers in base to use the new ones, and also the other files I happened to touch. This splits out the stuff from scoped_handle into a few separate files. I just deleted ScopedFindFile since it was only used in one place and it wasn't even really helping there. I removed StackBstr which was a #define and used the "regular" ScopedBstr in the 7 places that used it. This is an optimization to avoid an extra allocation, but none of the callers are remotely performance critical. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3781009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62843 0039d316-1c4b-4281-b951-d872f2087c98
* Convert LOG(INFO) to VLOG(1) - net/.pkasting@chromium.org2010-10-1648-263/+243
| | | | | | | | | | 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
* Plumb up the preconnect logic from pools to StreamFactory.willchan@chromium.org2010-10-1613-98/+497
| | | | | | | | | BUG=54450 TEST=none Review URL: http://codereview.chromium.org/3750009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62827 0039d316-1c4b-4281-b951-d872f2087c98
* In a fake SSLInfo, set security_bits to -1 instead of 0 wtc@chromium.org2010-10-151-2/+6
| | | | | | | | | | | | because the encryption strength is unknown (0 means no encryption). R=finnur BUG=53366,13049 TEST=no unit test failures Review URL: http://codereview.chromium.org/3749003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62785 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
* FTP: make file downloads work when directory listing is restrictedphajdan.jr@chromium.org2010-10-152-34/+65
| | | | | | | | | BUG=56734 TEST=see bug Review URL: http://codereview.chromium.org/3775005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62714 0039d316-1c4b-4281-b951-d872f2087c98
* Return a distinct error from ERR_NAME_NOT_RESOLVED when the call to ↵eroman@chromium.org2010-10-153-5/+19
| | | | | | | | | | | | | | | 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
* Revert 62645 - Disk cache: Remove the request throttling experiment.rvargas@google.com2010-10-141-1/+10
| | | | | | | | | | | | | | | | This is intended for the beta/stable branch. BUG=none TEST=none TBR=gavinp Review URL: http://codereview.chromium.org/3788002 TBR=rvargas@google.com Review URL: http://codereview.chromium.org/3775009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62655 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: More ctor/dtor cleanup.erg@google.com2010-10-143-1/+12
| | | | | | | | | | | (3.5 megs off debug linux .a files) BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3806005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62654 0039d316-1c4b-4281-b951-d872f2087c98
* Disk cache: Remove the request throttling experiment.rvargas@google.com2010-10-141-10/+1
| | | | | | | | | | | | | This is intended for the beta/stable branch. BUG=none TEST=none TBR=gavinp Review URL: http://codereview.chromium.org/3788002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62645 0039d316-1c4b-4281-b951-d872f2087c98
* Disk cache: Close the current experiment for new users.rvargas@google.com2010-10-141-28/+6
| | | | | | | | | | | | | (increasing the number of deleted entries) BUG=none TEST=none TBR=gavinp Review URL: http://codereview.chromium.org/3778002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62643 0039d316-1c4b-4281-b951-d872f2087c98
* Add the "Use SSL 3.0" and "Use TLS 1.0" checkboxes towtc@chromium.org2010-10-143-4/+49
| | | | | | | | | | | | the Options menu ("Under the Hood" tab) on Windows. R=mattm BUG=59125 TEST=net_unittests.exe --gtest_filter=SSLConfigServiceWinTest.SetTest. Manually verify the Options menu has the two new checkboxes. Review URL: http://codereview.chromium.org/3779003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62634 0039d316-1c4b-4281-b951-d872f2087c98
* Add actual bytes sent/received to net-internals.mmenke@chromium.org2010-10-1416-40/+113
| | | | | | | | | BUG=54745 TEST=manual Review URL: http://codereview.chromium.org/3582007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62627 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 62605 - Make Run methods non-virtual on RRResolverWorker.cbentzel@chromium.org2010-10-141-2/+2
| | | | | | | | | | | | | | | | | | I don't think this needs to be reverted, but git cl dcommit went a bit wacky and I'm being conservative. The goal is to turn on -Wnon-virtual-dtor warning for gcc. In this case, the virtual Run()'s do not appear necessary, especially since they are private. If intended to be derived from, there is a "delete this" in the base class which would point to requiring a virtual destructor. BUG=45135 TEST=Target builds, validated that there are no derived classes of RRResolverWorker. Review URL: http://codereview.chromium.org/3824002 TBR=cbentzel@chromium.org Review URL: http://codereview.chromium.org/3813005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62611 0039d316-1c4b-4281-b951-d872f2087c98
* Make Run methods non-virtual on RRResolverWorker.cbentzel@chromium.org2010-10-141-2/+2
| | | | | | | | | | | | | The goal is to turn on -Wnon-virtual-dtor warning for gcc. In this case, the virtual Run()'s do not appear necessary, especially since they are private. If intended to be derived from, there is a "delete this" in the base class which would point to requiring a virtual destructor. BUG=45135 TEST=Target builds, validated that there are no derived classes of RRResolverWorker. Review URL: http://codereview.chromium.org/3824002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62605 0039d316-1c4b-4281-b951-d872f2087c98
* net: update DNSSEC chain embedded extension OID.agl@chromium.org2010-10-141-3/+4
| | | | | | | | | We now have an assigned OID in the Google corporate OID subspace. BUG=none TEST=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62593 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup StreamFactory && StreamRequest APIs.willchan@chromium.org2010-10-147-178/+122
| | | | | | | | | | | | Stop refcounting StreamRequest. Establish a clear ownership of the StreamRequest. Deletion implies cancellation. Use ScopedRunnableMethodFactory::NewRunnableMethod() instead of NewRunnableMethod(). Remove Start() from StreamRequest. This is an implementation detail of HttpStreamRequest, so it only exists there now. BUG=59103 TEST=existing, this is a pure refactor. Review URL: http://codereview.chromium.org/3746002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62581 0039d316-1c4b-4281-b951-d872f2087c98
* Add a RenewStreamForAuth method to HttpStream, replacing DetachConnectionvandebo@chromium.org2010-10-1413-93/+52
| | | | | | | | | BUG=58192 TEST=Start chrome with --auth-schemes=NTLM and --proxy-server pointing to a Microsoft Forefront Threat Management Gateway proxy configurated for Integrated Authentication. Assuming the user is part of the same domain as the proxy, authentication should work transparently, and the user should not be presented with auth prompts. Also, net_unittests should pass. Review URL: http://codereview.chromium.org/3676004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62564 0039d316-1c4b-4281-b951-d872f2087c98
* SSLConfig struct CertAndStatus is not initializing all its members.finnur@chromium.org2010-10-141-2/+2
| | | | | | | | | | | BUG=None TEST=None CID=13288 Review URL: http://codereview.chromium.org/3781002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62532 0039d316-1c4b-4281-b951-d872f2087c98
* Fix FTP directory listing for ftp.cisco.com.phajdan.jr@chromium.org2010-10-145-2/+45
| | | | | | | | | BUG=58963 TEST=net_unittests Review URL: http://codereview.chromium.org/3774002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62515 0039d316-1c4b-4281-b951-d872f2087c98
* Move Stats, histograms, and field trial into a metrics subdirectory of base andbrettw@chromium.org2010-10-1451-179/+198
| | | | | | | | | 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
* Add an option ProxyService::Create() to disable use of proxy auto-config.eroman@chromium.org2010-10-142-34/+77
| | | | | | | BUG=40797 Review URL: http://codereview.chromium.org/3646004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62487 0039d316-1c4b-4281-b951-d872f2087c98
* base: Move SplitString functions into the base namespace and update the callers.tfarina@chromium.org2010-10-1413-24/+24
| | | | | | | | | BUG=None TEST=trybots Review URL: http://codereview.chromium.org/3750001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62483 0039d316-1c4b-4281-b951-d872f2087c98
* Integrate the SpdyProxyClientSocket into the HttpStreamRequestrch@chromium.org2010-10-1317-338/+913
| | | | | | | | | | | to support fetching HTTPS URLS over a SPDY Proxy. BUG=29625 TEST=HttpNetworkTransactionTest.HttpsProxySpdyConnect Review URL: http://codereview.chromium.org/3417010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62468 0039d316-1c4b-4281-b951-d872f2087c98
* Fix clang build: remove a const on a return value.willchan@chromium.org2010-10-131-1/+1
| | | | | | | | | BUG=none TEST=builds Review URL: http://codereview.chromium.org/3768001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62370 0039d316-1c4b-4281-b951-d872f2087c98
* Add ClientSocketPoolBaseHelper support for preconnect.willchan@chromium.org2010-10-1313-37/+646
| | | | | | | | | | | | | | | | | | | Adds a RequestSockets() API to ClientSocketPool interface. - no RequestPriority param, all requests default to LOWEST. - adds a |num_sockets| param to control how many sockets to try to ensure are connected. Adds an implementation for said function in ClientSocketPoolBaseHelper. Adds a new ClientSocketPoolBaseHelper::Flag type to modify socket request behavior. In this case, we bypass idle sockets. Adds a preconnect concept to ConnectJob. This lets normal requests hijack preconnect jobs. Modifies all ClientSocketPool subclasses to support new RequestSockets API(). Adds new tests. No client actually uses this API yet. We need to plumb it up to the preconnect system. BUG=54450 TEST=new tests in ClientSocketPoolBaseTest Review URL: http://codereview.chromium.org/3689004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62365 0039d316-1c4b-4281-b951-d872f2087c98
* Remove non-internationalized SDCH warning message.jar@chromium.org2010-10-121-1/+5
| | | | | | | | BUG=33185 r=kmixter Review URL: http://codereview.chromium.org/3702005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62353 0039d316-1c4b-4281-b951-d872f2087c98