summaryrefslogtreecommitdiffstats
path: root/chrome/browser/net/http_pipelining_compatibility_client.cc
Commit message (Collapse)AuthorAgeFilesLines
* chrome: Update include paths of string_split.h to its new location.tfarina@chromium.org2013-02-281-1/+1
| | | | | | | | | | BUG=175186 TBR=brettw@chromium.org Review URL: https://chromiumcodereview.appspot.com/12378016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@185315 0039d316-1c4b-4281-b951-d872f2087c98
* Rename FilePath to base::FilePath and use new location of ↵brettw@chromium.org2013-02-081-1/+1
| | | | | | | | string_number_conversions in some chrome subdirectories. Review URL: https://codereview.chromium.org/12218065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181564 0039d316-1c4b-4281-b951-d872f2087c98
* Removed calls to URLRequest::URLRequest in favor of ↵erikwright@chromium.org2012-09-011-3/+3
| | | | | | | | | | | | | | | URLRequestContext::CreateRequest This patch is by departed intern shalev@chromium.org, but it failed to pass the CQ for reasons unrelated to the patch. Original review at http://codereview.chromium.org/10870060/ . TBR=mmenke@chromium.org,michaeln@chromium.org,brettw@chromium.org,ben@chromium.org,zelidrag@chromium.org,phajdan.jr@chromium.org,pfeldman@chromium.org BUG=142945 Review URL: https://chromiumcodereview.appspot.com/10910044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154550 0039d316-1c4b-4281-b951-d872f2087c98
* Added URLRequestContext to constructor for URLRequest.erikwright@chromium.org2012-06-221-2/+3
| | | | | | | | | | | | | Previously, the context was set using the member function set_context. Note that the context for a URLRequest is not allowed to be null. BUG=81979 TEST=None TBR=ajwong,brettw,vitalybuka,sky Review URL: https://chromiumcodereview.appspot.com/10559036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143595 0039d316-1c4b-4281-b951-d872f2087c98
* Stop refcounting URLRequestContext.willchan@chromium.org2012-05-111-1/+1
| | | | | | | | | | | | While doing so, fix a few issues with the code like ordering of URLRequestContext to ensure correct destruction order. Also fix const correctness in some places. BUG=58859 TEST=none TBR=willchan Review URL: https://chromiumcodereview.appspot.com/10299002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136497 0039d316-1c4b-4281-b951-d872f2087c98
* Expand pipelining trial to 100% of dev channel.simonjam@chromium.org2012-04-281-3/+3
| | | | | | | | | | BUG=110794 TEST=None Review URL: http://codereview.chromium.org/10170044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134439 0039d316-1c4b-4281-b951-d872f2087c98
* Fix HttpPipeliningCompatibilityClient ownership.willchan@chromium.org2012-04-181-15/+9
| | | | | | | | | | | | | | Previously it was unowned. Now IOThread::Globals owns it. Due to this, move the CHECK for no leaking URLRequests to the IOThread::Globals destructor. Also, fix up some lifetime ordering issues with HttpPipeliningCompatibilityClient. This lets us delete the HttpPipeliningCompatibilityClient synchronously from within the finishing callback. BUG=123830 TEST=none Review URL: http://codereview.chromium.org/10041008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132715 0039d316-1c4b-4281-b951-d872f2087c98
* Enable HTTP pipelining field trial on dev channel for 10% of users.simonjam@chromium.org2012-04-161-0/+2
| | | | | | | | | BUG=110794 TEST=None Review URL: https://chromiumcodereview.appspot.com/10105002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132444 0039d316-1c4b-4281-b951-d872f2087c98
* Delete pipelining canary request when finished, so it doesn't continue tosimonjam@chromium.org2012-04-121-4/+4
| | | | | | | | | | | | | callback. Also, re-enable the pipelining trial for another week. BUG=122167 TEST=None Review URL: https://chromiumcodereview.appspot.com/10068023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132049 0039d316-1c4b-4281-b951-d872f2087c98
* Field Trials choices can now be forced from a command line argument, and yet ↵mad@chromium.org2012-04-121-2/+2
| | | | | | | | | | | | | | | | | still behave as if a coin was tossed, except the coin is tricked :-). To do this, we needed to change the usage from a regular constructor (which is not private) to a static CreateInstance method (which is why there is a bunch of TBR'd owners, those changes were trivial to existing users of FieldTrials). OWNERs of trivially changed files: TBR=sky,jamesr,cpu,joi, BUG=119726 TEST=base_unittests.exe --gtest_filter=FieldTrialTest.* You can also find an active field trial name and force it to a given group by passing the following command line argument "--force-fieldtest=<trial_name>/<default_group_name>/<group_name>/" Review URL: http://codereview.chromium.org/9705074 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131948 0039d316-1c4b-4281-b951-d872f2087c98
* Temporarily disable pipelining field trial.simonjam@chromium.org2012-04-091-1/+2
| | | | | | | | | BUG=122167 TEST=None Review URL: https://chromiumcodereview.appspot.com/10031006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131424 0039d316-1c4b-4281-b951-d872f2087c98
* Renew pipelining field trial.simonjam@chromium.org2012-04-041-2/+2
| | | | | | | | | BUG=110794 TEST=None Review URL: https://chromiumcodereview.appspot.com/9978009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130700 0039d316-1c4b-4281-b951-d872f2087c98
* Add a canary request to the pipelining field trial.simonjam@chromium.org2012-04-021-158/+314
| | | | | | | | | BUG=110794 TEST=Ran official build. Review URL: https://chromiumcodereview.appspot.com/9866025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130170 0039d316-1c4b-4281-b951-d872f2087c98
* Asynchronously delete pipeline compatibility test.simonjam@chromium.org2012-03-301-2/+9
| | | | | | | | | | BUG=120983 TEST=Built and ran official Chrome Review URL: http://codereview.chromium.org/9935001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129808 0039d316-1c4b-4281-b951-d872f2087c98
* Bump pipelining compatibility test to 100% of canary channel.simonjam@chromium.org2012-03-291-1/+1
| | | | | | | | | BUG=110794 TEST=None Review URL: https://chromiumcodereview.appspot.com/9924019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129666 0039d316-1c4b-4281-b951-d872f2087c98
* Move pipeline test requests around a bit. Change to 0-index.simonjam@chromium.org2012-03-281-11/+11
| | | | | | | | | BUG=110794 TEST=Built and ran with official Chrome Review URL: https://chromiumcodereview.appspot.com/9869001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129527 0039d316-1c4b-4281-b951-d872f2087c98
* Bump pipelining experiment to 20% of canary channel.simonjam@chromium.org2012-03-271-1/+1
| | | | | | | | | | BUG=110794 TEST=None Review URL: http://codereview.chromium.org/9866016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129153 0039d316-1c4b-4281-b951-d872f2087c98
* Launch HTTP pipelining compatibility field trial for 1% of canary users.simonjam@chromium.org2012-03-221-6/+5
| | | | | | | | | | BUG=110794 TEST=Built official Chrome and let it run Review URL: http://codereview.chromium.org/9782002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@128281 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 127579 - Launch HTTP pipelining compatibility field trial for 1% of ↵simonjam@chromium.org2012-03-201-5/+6
| | | | | | | | | | | | | | | canary users. BUG=110794 TEST=Built official Chrome and let it run Review URL: http://codereview.chromium.org/9722019 TBR=simonjam@chromium.org Review URL: https://chromiumcodereview.appspot.com/9727013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127594 0039d316-1c4b-4281-b951-d872f2087c98
* Launch HTTP pipelining compatibility field trial for 1% of canary users.simonjam@chromium.org2012-03-201-6/+5
| | | | | | | | | | BUG=110794 TEST=Built official Chrome and let it run Review URL: http://codereview.chromium.org/9722019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127579 0039d316-1c4b-4281-b951-d872f2087c98
* Add a disabled FieldTrial for HTTP pipelining compatibility.simonjam@chromium.org2012-03-091-3/+105
| | | | | | | | | | BUG=110794 TEST=Ran modified build with this enabled. Review URL: http://codereview.chromium.org/9636001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125917 0039d316-1c4b-4281-b951-d872f2087c98
* Collect and record pipelining stats from test server.simonjam@chromium.org2012-03-071-1/+78
| | | | | | | | | | BUG=110794 TEST=unit_tests Review URL: http://codereview.chromium.org/9606003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125479 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure pipelined requests are sent in the same order they're queued. Also, ↵simonjam@chromium.org2012-03-031-24/+17
| | | | | | | | | | | | ensure HttpPipeliningCompatibilityClient closes the stream when it's done with a request. BUG=110794 TEST=net_unittests and unit_tests Review URL: http://codereview.chromium.org/9567025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124793 0039d316-1c4b-4281-b951-d872f2087c98
* Add a force pipelining option to load flags.simonjam@chromium.org2012-03-011-2/+17
| | | | | | | | | | | | | | | | | | | | | Details: - Add a HttpPipelinedHostForced class for connections with forced requests. + Forced requests get their own pipeline and there's only one per host. + They always try to pipeline and won't retry if evicted. + Only one HttpStreamFactoryImpl::Job runs for all requests to the same origin with forced pipelining. All requests will fail if that Job fails. - Track HttpPipelinedHosts with a Key. Right now that's origin and force-pipelining, but it might be expanded to include content type. - Add a BufferedWriteStreamSocket that wraps a normal socket. It buffers Write() calls until a task fires to dispatch the buffer to the underlying socket. BUG=110794 TEST=net_unittests and unit_tests Review URL: http://codereview.chromium.org/9433015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124487 0039d316-1c4b-4281-b951-d872f2087c98
* Add client for background testing of HTTP pipelining.simonjam@chromium.org2012-02-131-0/+188
| | | | | | | | | | | | | | | Note this isn't hooked up, so it won't affect users yet. Once the servers are up, it'll be hooked up to a field test, much like the UDP echo test. Also missing is a force-pipelining option for requests. I'll add that in a separate CL. BUG=110794 TEST=unit_tests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=121572 Review URL: http://codereview.chromium.org/9302024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@121766 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 121572 - Add client for background testing of HTTP pipelining.simonjam@chromium.org2012-02-111-188/+0
| | | | | | | | | | | | | | | | Note this isn't hooked up, so it won't affect users yet. Once the servers are up, it'll be hooked up to a field test, much like the UDP echo test. Also missing is a force-pipelining option for requests. I'll add that in a separate CL. BUG=110794 TEST=unit_tests Review URL: http://codereview.chromium.org/9302024 TBR=mmenke@chromium.org Review URL: https://chromiumcodereview.appspot.com/9383029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@121605 0039d316-1c4b-4281-b951-d872f2087c98
* Add client for background testing of HTTP pipelining.simonjam@chromium.org2012-02-111-0/+188
Note this isn't hooked up, so it won't affect users yet. Once the servers are up, it'll be hooked up to a field test, much like the UDP echo test. Also missing is a force-pipelining option for requests. I'll add that in a separate CL. BUG=110794 TEST=unit_tests Review URL: http://codereview.chromium.org/9302024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@121572 0039d316-1c4b-4281-b951-d872f2087c98