summaryrefslogtreecommitdiffstats
path: root/net/http
Commit message (Collapse)AuthorAgeFilesLines
* Fix comment in http_stream_request.cc.willchan@chromium.org2011-01-261-2/+2
| | | | | | | | | BUG=66472 TEST=none Review URL: http://codereview.chromium.org/6313014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72604 0039d316-1c4b-4281-b951-d872f2087c98
* Prevent over-preconnecting when we already have a SpdySession.willchan@chromium.org2011-01-262-5/+41
| | | | | | | | | | | Note, this isn't a complete fix for the bug. We need to persist information for that. BUG=66472 TEST=none Review URL: http://codereview.chromium.org/6340016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72600 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Clean up net unit testing code." since it breaks the shlib builder.erg@google.com2011-01-258-315/+213
| | | | | | | | | | This reverts commit 006f377bbb65ece3490b7c76e58e0dc4cb330909 (r72562). BUG=68682 TEST=compiles TBR=thomasvl git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72564 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up net unit testing code.erg@google.com2011-01-258-213/+315
| | | | | | | | | | | | | | | | | - Move code included from blah_unittest.h (where blah_unittest.cc has actual unittests) into their own files, often completely out-of-lining the definitions. - Remove anonymous namespaces from headers. - Reorder method declarations. - Make other unit test link against net_test_support instead of reaching in and including .h files in their targets directly. BUG=68682 TEST=compiles Review URL: http://codereview.chromium.org/6264013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72562 0039d316-1c4b-4281-b951-d872f2087c98
* Prototype of chunked transfer encoded POST.satish@chromium.org2011-01-255-14/+48
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6134003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72471 0039d316-1c4b-4281-b951-d872f2087c98
* More net/ reordering.erg@google.com2011-01-214-52/+52
| | | | | | | | | | | | | | In addition to the normal method reordering, this patch also deinlines net/base/test_completion_callback.h and places the compiled code in the net_test_support target. Minimization of that header also required adding includes in a few unit tests. BUG=68682 TEST=compiles Review URL: http://codereview.chromium.org/6341004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72162 0039d316-1c4b-4281-b951-d872f2087c98
* Honor User-Agent header added by WebCorecaseq@google.com2011-01-213-0/+21
| | | | | | | | | | | | | - do not strip User-Agent while flattening headers in renderer - do not override User-Agent header in the network stack if one is already supplied by higher level; - added RequestHeaders::SetHeaderIfMissing TEST=URLRequestTestHTTP.OverrideUserAgent BUG=http://crbug.com/67063 Review URL=http://codereview.chromium.org/6065008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72131 0039d316-1c4b-4281-b951-d872f2087c98
* Remove obsolete base/lock.h and fix up callers to use the new header file andbrettw@chromium.org2011-01-211-1/+0
| | | | | | | | | | | 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-207-147/+143
| | | | | | | | | 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
* Remove unnecessary const from HttpAuthCache to appease the clang gods.cbentzel@chromium.org2011-01-131-1/+1
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/6303001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71321 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Use AUTH_SCHEME enum instead of a string.cbentzel@chromium.org2011-01-1317-154/+209
| | | | | | | | | BUG=None TEST=trybots Review URL: http://codereview.chromium.org/6191001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71318 0039d316-1c4b-4281-b951-d872f2087c98
* Address post-review/commit nits from wtc for r71096 ( ↵rsleevi@chromium.org2011-01-132-5/+6
| | | | | | | | | | | http://codereview.chromium.org/6017010/ ) and r71071 ( http://codereview.chromium.org/6120002/ ) BUG=none TEST=none Review URL: http://codereview.chromium.org/6221004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71270 0039d316-1c4b-4281-b951-d872f2087c98
* Remove extraneous #include's from http_auth_controller.cccbentzel@chromium.org2011-01-121-2/+0
| | | | | | | | | BUG=None TEST=trybots Review URL: http://codereview.chromium.org/6109009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71164 0039d316-1c4b-4281-b951-d872f2087c98
* net: Disable False Start and clear the SSL client auth cache for HTTPS ↵agl@chromium.org2011-01-112-0/+117
| | | | | | | | | | | | | | | | | | | | | proxies on failure. When performing TLS client auth with an HTTPS proxy, disable TLS false start to better handle SSL handshake failures, such as the HTTPS proxy requiring a client certificate. In addition, when an HTTPS proxy fails, ensure that it is removed from the SSL client auth cache, so that if the failure was due to an invalid client certificate, the user can be prompted to select one again. Depends on: http://codereview.chromium.org/6017010 Patch by: Ryan Sleevi BUG=66424 TEST=HttpNetworkTransactionTest.Proxy_ClientAuthCertCache git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71096 0039d316-1c4b-4281-b951-d872f2087c98
* net: Ensure that when using False Start + client auth, bad client ↵agl@chromium.org2011-01-115-62/+260
| | | | | | | | | | | | | | | | | | | | | | | certificates are not cached. If an SSL handshake fails when client certificates are used, ensure that the client certificate selected is removed from the SSL client auth cache. This ensures that the user is prompted to select a certificate again, as the cause of the failure may have been due to selecting the wrong certificate or selecting no certificate when one is required. The existing logic worked when TLS False Start was disabled, but could fail when False Start was used or when the peer requests renegotiation. This changes ensures the client certificate is removed from the cache by moving the cache removal layer from the HttpStreamRequest to the HttpNetworkTransaction. Patch by: Ryan Sleevi BUG=66424 TEST=HttpNetworkTransactionTest.ClientAuthCertCache* git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71071 0039d316-1c4b-4281-b951-d872f2087c98
* More reordering the methods in headers in net/.erg@chromium.org2011-01-116-94/+96
| | | | | | | | | BUG=68682 TEST=compiles Review URL: http://codereview.chromium.org/6186005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71017 0039d316-1c4b-4281-b951-d872f2087c98
* Add a port to the SPDY debugging flags so that we can force SPDY for just ambelshe@chromium.org2011-01-085-6/+59
| | | | | | | | | | | | single port without forcing SPDY on all ports. BUG=none TEST=none --use-spdy="ssl,port=1000" Review URL: http://codereview.chromium.org/6083012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70834 0039d316-1c4b-4281-b951-d872f2087c98
* Start reordering the methods in headers in net/.erg@google.com2011-01-075-41/+45
| | | | | | | | | | | | This patch also starts reordering some of the cc files to match their headers. More of both cleanups will be done in future patches. BUG=68682 TEST=compiles Review URL: http://codereview.chromium.org/6085013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70799 0039d316-1c4b-4281-b951-d872f2087c98
* Remove auth token generation timing histograms.cbentzel@chromium.org2011-01-072-29/+0
| | | | | | | | | | | These didn't turn out to be terribly useful, and add some complexity to HttpAuthHandler. BUG=NONE TEST=targets build Review URL: http://codereview.chromium.org/6103003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70740 0039d316-1c4b-4281-b951-d872f2087c98
* reapply r70628. It was wrongly blamed for breaking Linux valgrind bots.estade@chromium.org2011-01-076-47/+124
| | | | | | | | | | | | | | | | | | | | | | | | | The revert was r70688. original author: Matt Menke First pass at adding http/backend cache events to the NetLog. Adds sources and events for ActiveCacheEntry and EntryImpl objects, as well as adding cache read/write events to HttpCacheTransactions. Most new read/write events are only logged when NetLog logging mode is set to log all events. Also, net-internals now merges begin and end events that have parameters, as long as only one of them has parameters. I think this increases readability, at the cost of making it a little more difficult to follow timings with just the "st" values. BUG=59382 TEST=none yet, other than updates to existing tests. Review URL: http://codereview.chromium.org/6150002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70716 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r70628. It broke Linux valgrind bots. See for exampleestade@chromium.org2011-01-076-124/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | http://build.chromium.org/p/chromium.memory/builders/Linux%20Tests%20%28valgrind%29%281%29/builds/963 First pass at adding http/backend cache events to the NetLog. Adds sources and events for ActiveCacheEntry and EntryImpl objects, as well as adding cache read/write events to HttpCacheTransactions. Most new read/write events are only logged when NetLog logging mode is set to log all events. Also, net-internals now merges begin and end events that have parameters, as long as only one of them has parameters. I think this increases readability, at the cost of making it a little more difficult to follow timings with just the "st" values. BUG=59382 TEST=none yet, other than updates to existing tests. Originally applied: http://src.chromium.org/viewvc/chrome?view=rev&revision=70618 Reverted: http://src.chromium.org/viewvc/chrome?view=rev&revision=70619 Fixed and trying again... Review URL: http://codereview.chromium.org/6125001 BUG=none TEST=linux valgrind goes green. Review URL: http://codereview.chromium.org/6142003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70688 0039d316-1c4b-4281-b951-d872f2087c98
* First pass at adding http/backend cache events to the NetLog. mmenke@chromium.org2011-01-066-47/+124
| | | | | | | | | | | | | | | | | | | | | | | | | Adds sources and events for ActiveCacheEntry and EntryImpl objects, as well as adding cache read/write events to HttpCacheTransactions. Most new read/write events are only logged when NetLog logging mode is set to log all events. Also, net-internals now merges begin and end events that have parameters, as long as only one of them has parameters. I think this increases readability, at the cost of making it a little more difficult to follow timings with just the "st" values. BUG=59382 TEST=none yet, other than updates to existing tests. Originally applied: http://src.chromium.org/viewvc/chrome?view=rev&revision=70618 Reverted: http://src.chromium.org/viewvc/chrome?view=rev&revision=70619 Fixed and trying again... Review URL: http://codereview.chromium.org/6125001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70628 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 70618 - First pass at adding http/backend cache events to the NetLog. mmenke@chromium.org2011-01-066-124/+47
| | | | | | | | | | | | | | | | | | | | | | | Adds sources and events for ActiveCacheEntry and EntryImpl objects, as well as adding cache read/write events to HttpCacheTransactions. Most new read/write events are only logged when NetLog logging mode is set to log all events. Also, net-internals now merges begin and end events that have parameters, as long as only one of them has parameters. I think this increases readability, at the cost of making it a little more difficult to follow timings with just the "st" values. BUG=59382 TEST=none yet, other than updates to existing tests. Review URL: http://codereview.chromium.org/4067002 TBR=mmenke@chromium.org Review URL: http://codereview.chromium.org/6005015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70619 0039d316-1c4b-4281-b951-d872f2087c98
* First pass at adding http/backend cache events to the NetLog. mmenke@chromium.org2011-01-066-47/+124
| | | | | | | | | | | | | | | | | | | | Adds sources and events for ActiveCacheEntry and EntryImpl objects, as well as adding cache read/write events to HttpCacheTransactions. Most new read/write events are only logged when NetLog logging mode is set to log all events. Also, net-internals now merges begin and end events that have parameters, as long as only one of them has parameters. I think this increases readability, at the cost of making it a little more difficult to follow timings with just the "st" values. BUG=59382 TEST=none yet, other than updates to existing tests. Review URL: http://codereview.chromium.org/4067002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70618 0039d316-1c4b-4281-b951-d872f2087c98
* Move non_thread_safe from base to base/threading and into the base namespace.brettw@chromium.org2011-01-015-10/+11
| | | | | | | | TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6005010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70351 0039d316-1c4b-4281-b951-d872f2087c98
* Move base/thread.h to base/threading, fix up callers to use the new location.brettw@chromium.org2011-01-011-1/+1
| | | | | | | | TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6028009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70345 0039d316-1c4b-4281-b951-d872f2087c98
* Move platform_thread to base/threading and put in the base namespace. I left abrettw@chromium.org2010-12-311-2/+4
| | | | | | | | | | | stub and "using" declarations in the old location to avoid having to change the entire project at once. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6001010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70342 0039d316-1c4b-4281-b951-d872f2087c98
* Http cache: Remove a histogram.rvargas@google.com2010-12-282-46/+13
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6021009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70240 0039d316-1c4b-4281-b951-d872f2087c98
* Clobber the state of the next-protocol-negotiation when disabling spdy.mbelshe@chromium.org2010-12-211-0/+2
| | | | | | | | | | | | | | | | We basically have two sets of config for SPDY - one is a general enablement flag, the other is the next-protocol advertisement. We previously could configure to advertise SPDY over NPN, but have SPDY disabled; this would lead to the bug with broken 'download' when accessing a SPDY page. BUG=66891 TEST=none Review URL: http://codereview.chromium.org/6007002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69891 0039d316-1c4b-4281-b951-d872f2087c98
* Follow up to r69305agl@chromium.org2010-12-171-1/+1
| | | | | | | | | | | Addressing last comment by willchan. BUG=63867 TEST=Navigate to an HTTPS page with Snap Start running and no network connection. http://codereview.chromium.org/5826001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69574 0039d316-1c4b-4281-b951-d872f2087c98
* Allow a non-200 (or non-407) response for a CONNECT request from an HTTPS proxyrch@chromium.org2010-12-1714-27/+389
| | | | | | | | | | | | | | | | | to be consumed by chrome. Among other things, this will allow the proxy to inform the user that the hostname could not be resolved or similar conditions. This adds a new OnHttpsProxyTunnelConnectionResponse method to StreamRequest::Delegate which is invoked when an HTTPS proxy returns a non-200, non-407 response. The method is called with an HttpResponseInfor argument to access the request headers, and an HttpStream argument to access the response body. BUG=none TEST=HttpNetworkTransactionTest Review URL: http://codereview.chromium.org/4935001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69513 0039d316-1c4b-4281-b951-d872f2087c98
* Almost done deinlining virtual methods.erg@google.com2010-12-161-0/+4
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5841002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69470 0039d316-1c4b-4281-b951-d872f2087c98
* Cache certificate verification results in memory.wtc@google.com2010-12-1612-12/+62
| | | | | | | | | R=agl BUG=63357 TEST=none Review URL: http://codereview.chromium.org/5386001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69414 0039d316-1c4b-4281-b951-d872f2087c98
* net: fix callbacks in DiskCacheBasedSSLHostInfoagl@chromium.org2010-12-152-28/+79
| | | | | | | | | | | | | | | | | | | | Previously, DiskCacheBasedSSLHostInfo had a couple of problems: * I had assumed that the disk cache was taking references to the callback. It wasn't. * Even if it were, I was passing pointers into DiskCacheBasedSSLHostInfo which needed to be live for the duration of the callback. This change switches to a custom callback class which doesn't need reference counting and which contains pointer members that remain live for the duration of the callback. BUG=63867 TEST=Navigate to an HTTPS page with Snap Start running and no network connection. http://codereview.chromium.org/5826001/ git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69305 0039d316-1c4b-4281-b951-d872f2087c98
* Temporarily allow IO when loading shared GSSAPI librarycbentzel@chromium.org2010-12-141-0/+5
| | | | | | | | | | | BUG=66702 TEST=none Contributed by: asanka@google.com Review URL: http://codereview.chromium.org/5716004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69148 0039d316-1c4b-4281-b951-d872f2087c98
* Even more virtual method deinlining.erg@google.com2010-12-136-16/+30
| | | | | | | | | 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
* Histogram target for each HTTP Authentication Scheme.cbentzel@chromium.org2010-12-139-12/+101
| | | | | | | | | | | | | For each HTTP authentication scheme, record whether it was started for a proxy, a secure proxy, a server, or a secure server. Also, use a new scheme-based enumerated value as an index rather than the score. BUG=None TEST=trybots, look at about:histograms after authenticating to different resources. Review URL: http://codereview.chromium.org/5563006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69018 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 68696 - Disable SPDY by default as a test to see if heap corruption ↵willchan@chromium.org2010-12-092-5/+1
| | | | | | | | | | | | | | | | goes away. Will re-enable after a canary goes out with this off. BUG=65948 TEST=none Review URL: http://codereview.chromium.org/5592009 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/5727001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68747 0039d316-1c4b-4281-b951-d872f2087c98
* Start deinlining non-empty virtual methods. (This will be automatically checkederg@google.com2010-12-094-7/+25
| | | | | | | | | | | for in the future.) BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5574006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68746 0039d316-1c4b-4281-b951-d872f2087c98
* Disable SPDY by default as a test to see if heap corruption goes away.willchan@chromium.org2010-12-092-1/+5
| | | | | | | | | | | Will re-enable after a canary goes out with this off. BUG=65948 TEST=none Review URL: http://codereview.chromium.org/5592009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68696 0039d316-1c4b-4281-b951-d872f2087c98
* Http cache: Remove the option to disable byte range support.rvargas@google.com2010-12-095-79/+12
| | | | | | | | | | BUG=58323 TEST=current tests Review URL: http://codereview.chromium.org/5522014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68689 0039d316-1c4b-4281-b951-d872f2087c98
* Add the "virtual" keyword on method overrides that are missing it.erg@google.com2010-12-084-6/+6
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5648004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68606 0039d316-1c4b-4281-b951-d872f2087c98
* Initialize HttpStreamRequest::alternate_protocol_ to avoid Valgrind false ↵timurrrr@chromium.org2010-12-072-0/+2
| | | | | | | | | | reports BUG=64918 TEST=linux_valgrind trybot Review URL: http://codereview.chromium.org/5599004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68456 0039d316-1c4b-4281-b951-d872f2087c98
* Change HttpProxyClientSocketPool so that when a new SpdyStream isrch@chromium.org2010-12-031-1/+2
| | | | | | | | | | | | | | | | | | | | | created, it uses the NetLog from the SpdySession, not from the ConnectJob, so that we avoid circular dependencies. I attempted to verify the buggy behavior and the fix by adding a DCHECK() to ConnectJob::set_socket (which is the source of the cycle you detected), but unfortunately, it appears that our unit tests seems to use the same NetLog everywhere. So to test this, I added a very specific DCHECK in passive_log_collector.cc which failed on every HTTPS request through a SPDY proxy when using the old code, and which never failed using this new code. BUG=65217 TEST=none Review URL: http://codereview.chromium.org/5587005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68209 0039d316-1c4b-4281-b951-d872f2087c98
* Http cache: Make sure that we don't forget that an entry is sparservargas@google.com2010-12-033-15/+28
| | | | | | | | | | | | | | | | | | after we return the headers to the caller. We used the response code as the only indicator to know if an entry was sparse or not, but the response code is modified on the request if a byte-range was not requested... so for subsequent sub-ranges we could consider this a regular, non sparse resource. BUG=62460 TEST=net_unittests Review URL: http://codereview.chromium.org/5541004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68194 0039d316-1c4b-4281-b951-d872f2087c98
* HttpAuthHandlerDigest unit test cleanup.cbentzel@chromium.org2010-12-021-78/+79
| | | | | | | | | | | | DCHECK replaced by EXPECT for HttpAuthHandlerDigest unittests. Also, changed RespondToChallenge to return a success boolean and LOG the failure reasons, rather than returning an error string. BUG=None TEST=net_unittests --gtest_filter="*HttpAuthHandlerDigest*" Review URL: http://codereview.chromium.org/5205005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68006 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes the remaining unit tests failures for OpenSSL:joth@chromium.org2010-12-021-3/+12
| | | | | | | | | | | | | | | | - implements basic client certificate support in ssl socket - adds special-case IP address support to allow SSL connections to the test server (iff there is a trusted certificate in the store with 127.0.0.1 in its name) - enables the test server for loading the temporary cert - implements the DES encryptor (removed TODO about refactoring the file layout as it's already covered by a TODO in the .h file) - disabled KeygenHandler tests, as this is not implemented for openssl - disables the (firefox) importer unittests. BUG=None TEST=net_unittests now run green Review URL: http://codereview.chromium.org/5195001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67990 0039d316-1c4b-4281-b951-d872f2087c98
* Fix case where a SPDY stream with an un-verified cert would kill the browsermbelshe@google.com2010-12-011-1/+6
| | | | | | | | | | | | | | | if an https:// URL was attempted to be fetched across it. Also fix a bug where when two SSL connections are made, but only one is moved into a SPDY session (because the second is redundant), close the redundant connection. This had been leaking a un-verified SSL connection into the pool, which could then get re-used as though it was verified. BUG=64861 TEST=none Review URL: http://codereview.chromium.org/5409004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67869 0039d316-1c4b-4281-b951-d872f2087c98
* Update NetLog to be threadsafe.mmenke@chromium.org2010-12-013-52/+77
| | | | | | | | | | | | | | | | | | The ChromeNetLog is now owned by the browser process, and passed to the IOThread on creation. NetLog entries can be added from any thread. Observers must be able to handle having log entries added from any thread. Observers can add/remove themselves on any thread. BUG=63334 TEST=None, yet Review URL: http://codereview.chromium.org/4118004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67851 0039d316-1c4b-4281-b951-d872f2087c98
* Support predictive request for multiple preconnectionsjar@chromium.org2010-11-302-4/+8
| | | | | | | | | | | | | | | [reland CL 5271002... with CrOS / ARM interface update] Connected up to the API for requesting more than one preconnect at a time (that API also explicitly tags requsets as preconnects, so that they can be merged with actual navication requests). BUG=64246 r=willchan,mbelshe Review URL: http://codereview.chromium.org/5400002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67654 0039d316-1c4b-4281-b951-d872f2087c98