summaryrefslogtreecommitdiffstats
path: root/net/spdy
Commit message (Collapse)AuthorAgeFilesLines
* [SPDY] Replace SpdyIOBuffer with new SpdyBuffer classakalin@chromium.org2013-04-1731-413/+758
| | | | | | | | | | | | | | | | | | | Use SpdyBuffer for both SPDY reads and writes. A future CL will add hooks to SpdyBuffer so that we keep track of flow control windows properly. Replace SpdyFrameProducer with SpdyBufferProducer. Also introduce new SpdyReadQueue class for delegates of SpdyStream to use. BUG=176592 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=194560 Review URL: https://codereview.chromium.org/13990005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194564 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 194560 "[SPDY] Replace SpdyIOBuffer with new SpdyBuffer c..."akalin@chromium.org2013-04-1731-758/+414
| | | | | | | | | | | | | | | | | | | | | | | > [SPDY] Replace SpdyIOBuffer with new SpdyBuffer class > > Use SpdyBuffer for both SPDY reads and writes. A future > CL will add hooks to SpdyBuffer so that we keep track of > flow control windows properly. > > Replace SpdyFrameProducer with SpdyBufferProducer. > > Also introduce new SpdyReadQueue class for delegates > of SpdyStream to use. > > BUG=176592 > > Review URL: https://codereview.chromium.org/13990005 TBR=akalin@chromium.org Review URL: https://codereview.chromium.org/13996009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194562 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Replace SpdyIOBuffer with new SpdyBuffer classakalin@chromium.org2013-04-1731-414/+758
| | | | | | | | | | | | | | | | | Use SpdyBuffer for both SPDY reads and writes. A future CL will add hooks to SpdyBuffer so that we keep track of flow control windows properly. Replace SpdyFrameProducer with SpdyBufferProducer. Also introduce new SpdyReadQueue class for delegates of SpdyStream to use. BUG=176592 Review URL: https://codereview.chromium.org/13990005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194560 0039d316-1c4b-4281-b951-d872f2087c98
* Re-land greedy read support for SpdySessionrtenneti@chromium.org2013-04-176-87/+939
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added Unit test to close the socket during spdy_session's read and verifying that getting new URLs from the same network session don't result in a failure. Modified mock socket code to close the connection if mock_read closes the connection (because of ERR_CONNECTION_CLOSED error). Implement greedy approach to read all the data and process it from the ClientSocket until we block. Implemented the greedy approach to improves the network stack performance on the mobile. spdy_session_test_util.* files have the common code between SpdySessionSpd2Test and SpdySessionSpd3Test. Created this file to reduce errors and to avoid duplicating of code. Review URL: https://chromiumcodereview.appspot.com/11644088 The change adds a scoped_refptr to SpdySession in DoLoop to clean up the state and to finish pending reads, in case SpdyFramer receives GOAway and closes all streams. DoReadComplete was holding the last reference before and DoLoop was accessing the object after DoReadComplete deleted the object at the end of the function. Review URL: https://codereview.chromium.org/12212102 Added unit tests for use after free of SpdySession and unit test to test yielding + async during Read. A test that crashes when scoped_refptr to SpdySession deleted from DoLoop. Added the following test per rch@ to test interactions of yielding + async during Read. Do the following MockReads and verify all the data is consumed and SpdySession has yielded during Read. SYNCHRONOUS 8K SYNCHRONOUS 8K SYNCHRONOUS 8K SYNCHRONOUS 2K ASYNC 8K SYNCHRONOUS 8K SYNCHRONOUS 8K SYNCHRONOUS 8K SYNCHRONOUS 2K Review URL: https://chromiumcodereview.appspot.com/12207122 Reverted URL: https://chromiumcodereview.appspot.com/13608003/ BUG=175574, 166958, 224701, 175069 R=rch@chromium.org, jar@chromium.org TEST=network unit tests Review URL: https://codereview.chromium.org/13834009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194540 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Plumb SPDY frame type through write queueakalin@chromium.org2013-04-1525-335/+592
| | | | | | | | | | | | | | | | | | | | | | This lets us remove the waiting_completions_ queue from SpdyStream, and also lets us guarantee that a stream is activated only when SYN_STREAM is sent. Change some SpdyStream::Delegate function signatures. Also add SpdySendStatus to replace some booleans. Change SpdyStream::send_bytes_ to only track data frames. Add GetFrame{Min,Max}imumSize() function to SpdyFramer. Fix calculation of frame overhead that is buggy with SPDY/4 and add regression test. BUG=176592,231003 Review URL: https://chromiumcodereview.appspot.com/13977002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194127 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Refactor SpdySession's write queueakalin@chromium.org2013-04-1312-470/+580
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is in preparation for replacing the various IOBuffers used for reads/writes with a single SpdyBuffer class. Replace the priority queue of SpdyIOBufferProducers with a SpdyWriteQueue object, which is an an array of FIFO queues binned by priority. The priority queue was looking only at priority and so was not guaranteeing FIFO behavior among producers with the same priority. Remove the frame queue in SpdyStream and instead have it use the session's write queue directly. Remove unused fields from SpdyIOBuffer and clean it up. Propagate and handle errors from SpdyCredentialBuilder::Build. Rename SpdyIOBufferProducer to SpdyFrameProducer, have it return a SpdyFrame, clean up its interface, and move the stream-activating logic out of it. Replace uses of std::list with std::deque. Steamline logic in SpdySession that deals with the write queue. Convert some raw pointers to scoped_ptr<>. Convert a use of Unretained() in SpdySession to use the weak pointer factory. BUG=176582 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192975 Review URL: https://chromiumcodereview.appspot.com/13009012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194102 0039d316-1c4b-4281-b951-d872f2087c98
* net: Update the include paths of base/string_piece.h to its new location.tfarina@chromium.org2013-04-139-11/+11
| | | | | | | | | | | string_piece.h was moved into base/strings/ in r191206 - https://chromiumcodereview.appspot.com/12982018/ TBR=eroman@chromium.org,brettw@chromium.org Review URL: https://codereview.chromium.org/14223008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194055 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: fix expectations for USE_SYSTEM_ZLIBphajdan.jr@chromium.org2013-04-121-4/+4
| | | | | | | | | | | | | | Tested with: - zlib-1.2.7 on 64-bit Gentoo Linux - zlib-1.2.3 on 64-bit Gentoo Linux - zlib-1.2.3.4 on 64-bit Ubuntu Precise BUG=none Review URL: https://codereview.chromium.org/14087007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194016 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Add flag and about:flags entry for SPDY/4 alpha 1akalin@chromium.org2013-04-129-10/+75
| | | | | | | | BUG=230124 Review URL: https://codereview.chromium.org/13845012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193832 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 192975 "[SPDY] Refactor SpdySession's write queue"tyoshino@chromium.org2013-04-1112-574/+469
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Looks like this caused use after free on Chromium OS (amd64) ASAN bot. BUG=230259 > [SPDY] Refactor SpdySession's write queue > > This is in preparation for replacing the various IOBuffers used for reads/writes > with a single SpdyBuffer class. > > Replace the priority queue of SpdyIOBufferProducers with a SpdyWriteQueue object, > which is an an array of FIFO queues binned by priority. > The priority queue was looking only at priority and so was not guaranteeing > FIFO behavior among producers with the same priority. > > Remove the frame queue in SpdyStream and instead have it use the session's > write queue directly. > > Remove unused fields from SpdyIOBuffer and clean it up. > > Propagate and handle errors from SpdyCredentialBuilder::Build. > > Rename SpdyIOBufferProducer to SpdyFrameProducer, have it return a SpdyFrame, > clean up its interface, and move the stream-activating logic out of it. > > Replace uses of std::list with std::deque. > > Steamline logic in SpdySession that deals with the write queue. > > Convert some raw pointers to scoped_ptr<>. > > BUG=176582 > > Review URL: https://codereview.chromium.org/13009012 TBR=akalin@chromium.org Review URL: https://codereview.chromium.org/13880004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193597 0039d316-1c4b-4281-b951-d872f2087c98
* [net] Move host_cache files from net/base to net/dnsszym@chromium.org2013-04-102-2/+2
| | | | | | | | | | TBR=sky BUG=70818 Review URL: https://chromiumcodereview.appspot.com/13818025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193455 0039d316-1c4b-4281-b951-d872f2087c98
* Change the histogram in SpdySession::GetPeerAddress to look at the actual ↵rch@chromium.org2013-04-101-11/+14
| | | | | | | | | | | value return by the GetPeerAddress. BUG=224701 Review URL: https://chromiumcodereview.appspot.com/13862022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193298 0039d316-1c4b-4281-b951-d872f2087c98
* Convert SpdyControlType to SpdyFrameTypeakalin@chromium.org2013-04-0911-69/+93
| | | | | | | | | | | | | | | | | | | Remove NUM_CONTROL_FRAME_TYPES (used to indicate a data frame) and replace it with a DATA enum value. Also add {FIRST,LAST}_CONTROL_TYPE enum values and use them in checks. Strengthen error checking in spdy_framer.cc -- previously, converting to an enum type may inadvertently convert an invalid value to a valid one. This lands server change 45004245. Also take advantage of new enum values in some test functions. Review URL: https://codereview.chromium.org/13849003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193145 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite std::string("") to std::string(), Linux edition.dcheng@chromium.org2013-04-097-14/+16
| | | | | | | | | | | | | | | | | | | This patch was generated by running the empty_string clang tool across the Chromium Linux compilation database. Implicitly or explicitly constructing std::string() with a "" argument is inefficient as the caller needs to emit extra instructions to pass an argument, and the constructor needlessly copies a byte into internal storage. Rewriting these instances to simply call the default constructor appears to save ~14-18 kilobytes on an optimized release build. BUG=none Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=193020 Review URL: https://codereview.chromium.org/13145003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193040 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Rewrite std::string("") to std::string(), Linux edition."dcheng@chromium.org2013-04-097-16/+14
| | | | | | | | | | | | | | This reverts commit e59558b78e8c6a1b0bd916a724724b638c3c91b6. Revert "Fix build after r193020." This reverts commit 558a35897f6b3ffbcaefde927c1f150b815d140a. Revert "Really fix build after r193020." This reverts commit e3748a79b523a8d365d4a33ef986eebb4186fa78. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193030 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite std::string("") to std::string(), Linux edition.dcheng@chromium.org2013-04-097-14/+16
| | | | | | | | | | | | | | | | | This patch was generated by running the empty_string clang tool across the Chromium Linux compilation database. Implicitly or explicitly constructing std::string() with a "" argument is inefficient as the caller needs to emit extra instructions to pass an argument, and the constructor needlessly copies a byte into internal storage. Rewriting these instances to simply call the default constructor appears to save ~14-18 kilobytes on an optimized release build. BUG=none Review URL: https://codereview.chromium.org/13145003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193020 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Refactor SpdySession's write queueakalin@chromium.org2013-04-0913-469/+757
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is in preparation for replacing the various IOBuffers used for reads/writes with a single SpdyBuffer class. Replace the priority queue of SpdyIOBufferProducers with a SpdyWriteQueue object, which is an an array of FIFO queues binned by priority. The priority queue was looking only at priority and so was not guaranteeing FIFO behavior among producers with the same priority. Remove the frame queue in SpdyStream and instead have it use the session's write queue directly. Remove unused fields from SpdyIOBuffer and clean it up. Propagate and handle errors from SpdyCredentialBuilder::Build. Rename SpdyIOBufferProducer to SpdyFrameProducer, have it return a SpdyFrame, clean up its interface, and move the stream-activating logic out of it. Replace uses of std::list with std::deque. Steamline logic in SpdySession that deals with the write queue. Convert some raw pointers to scoped_ptr<>. BUG=176582 Review URL: https://codereview.chromium.org/13009012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192975 0039d316-1c4b-4281-b951-d872f2087c98
* Change the signature of SpdyFramer::SerializeNameValueBlockWithoutCompressionakalin@chromium.org2013-04-082-18/+19
| | | | | | | | This lands server change 44867738. Review URL: https://codereview.chromium.org/13529032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192945 0039d316-1c4b-4281-b951-d872f2087c98
* Fix indentation in spdy_framer_test.ccakalin@chromium.org2013-04-081-2/+1
| | | | | | | | This lands server change 44799980. Review URL: https://codereview.chromium.org/13454035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192933 0039d316-1c4b-4281-b951-d872f2087c98
* Add SpdyPriorityForest classakalin@chromium.org2013-04-082-0/+809
| | | | | | | | This lands server change 44685956. Review URL: https://codereview.chromium.org/13601029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192910 0039d316-1c4b-4281-b951-d872f2087c98
* Add helper function for SPDY 4akalin@chromium.org2013-04-082-0/+10
| | | | | | | | This lands server change 44568667. Review URL: https://codereview.chromium.org/13598037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192885 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite scoped_array<T> to scoped_ptr<T[]> in net, Linux edition.dcheng@chromium.org2013-04-076-14/+14
| | | | | | | | | | This changelist was automatically generated using a clang tool. BUG=171111 Review URL: https://codereview.chromium.org/13612004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192764 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up scoped_array<T> construction from NULL.dcheng@chromium.org2013-04-051-1/+1
| | | | | | | | | | This won't be allowed by unique_ptr<T> or scoped_ptr<T>. BUG=171118 Review URL: https://codereview.chromium.org/13710002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192518 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 181390 "SPDY - implement greedy approach to read all the ..."rtenneti@chromium.org2013-04-046-935/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert 181637 "SPDY - Small bug fix for user after free. Will ..." Revert 182012 "SPDY - Added unit tests for use after free of ..." > SPDY - implement greedy approach to read all the data and process it > from the ClientSocket until we block. > > This is more of a first cut at the greedy approach to see if it > improves the performance on the mobile. > > spdy_session_test_util.* files have the common code between > SpdySessionSpd2Test and SpdySessionSpd3Test. Will move other common > code in a separate CL. Created this file to reduce errors and > to avoid duplicating of code. > > R=willchan > BUG=166958 > TEST=network unittests > > Review URL: https://chromiumcodereview.appspot.com/11644088 > SPDY - Small bug fix for user after free. Will revert the > change asap. > > The change adds a scoped_refptr to SpdySession in DoLoop > to clean up the state and to finish pending reads, in case > SpdyFramer receives GOAway and closes all streams. > > DoReadComplete was holding the last reference before and > DoLoop was accessing the object after DoReadComplete deleted > the object at the end of the function. > > R=willchan@chromium.org > BUG=175069 > TEST=network unit tests > Review URL: https://codereview.chromium.org/12212102 > SPDY - Added unit tests for use after free of SpdySession > and unit test to test yielding + async during Read. > > A test that crashes when scoped_refptr to SpdySession deleted > from DoLoop. > > Added the following test per rch to test interactions of > yielding + async during Read. > > Do the following MockReads and verify all the data is consumed > and SpdySession has yielded during Read. > SYNCHRONOUS 8K > SYNCHRONOUS 8K > SYNCHRONOUS 8K > SYNCHRONOUS 2K > ASYNC 8K > SYNCHRONOUS 8K > SYNCHRONOUS 8K > SYNCHRONOUS 8K > SYNCHRONOUS 2K > > R=rch@chromium.org > BUG=175574,166958 > TEST=net unit tests > > Review URL: https://chromiumcodereview.appspot.com/12207122 TBR=rtenneti@chromium.org, rch@chromium.org Review URL: https://chromiumcodereview.appspot.com/13608003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192313 0039d316-1c4b-4281-b951-d872f2087c98
* Move version-agnostic functions from spdy_test_util_spdy[23] to _common.ttuttle@chromium.org2013-04-046-508/+258
| | | | | | | | | | | | | There are some functions in net/spdy/spdy_test_util_spdy[23] that are identical across the two files. Move them into net/spdy/spdy_test_util_common and include spdy_test_util_common.h in _spdy[23].h so tests can see them. BUG=226192 Review URL: https://chromiumcodereview.appspot.com/13582002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192262 0039d316-1c4b-4281-b951-d872f2087c98
* Remove experimental code to pick the "warmest" socketmmenke@chromium.org2013-03-303-11/+0
| | | | | | | | | | | | | | | | | | (based on age and bytes received) in favor of older algorithm to pick the most recently used socket. Tests showed no real performance difference, so defaulting to the older, simpler, and more intuitive algorithm. This is basically a revert of https://codereview.chromium.org/7251004 TBR=sergeyu@chromium.org BUG=222090 Review URL: https://codereview.chromium.org/12886034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191507 0039d316-1c4b-4281-b951-d872f2087c98
* net: extract net/cert out of net/basephajdan.jr@chromium.org2013-03-2912-12/+12
| | | | | | | | | | | | This introduces the following dependency of net/base on things outside: net/base/openssl_client_key_store.cc:#include "net/cert/x509_certificate.h" BUG=70818 Review URL: https://codereview.chromium.org/13006020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191450 0039d316-1c4b-4281-b951-d872f2087c98
* Convert net to use base::string16 instead of string16.brettw@chromium.org2013-03-292-4/+4
| | | | | | | | | | This is just a search-and-replace. BUG= Review URL: https://codereview.chromium.org/13329004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191432 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Remove some setters in SpdyStreamakalin@chromium.org2013-03-2511-43/+107
| | | | | | | | | | | | | | Instead pass parameters into SpdyStream's constructor. This simplifies SpdyStream, as we can now be sure that its priority is fixed. Add function to convert from SpdyPriority to RequestPriority, and use them to set the correct priority for push streams. Replace uses of Unretained() with weak pointers. BUG=176582 Review URL: https://codereview.chromium.org/12989038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@190481 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Add session flow control data to SPDY info in net-internalsakalin@chromium.org2013-03-231-0/+5
| | | | | | | | | | Also fix a typo. BUG=176592 Review URL: https://codereview.chromium.org/12459022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@190085 0039d316-1c4b-4281-b951-d872f2087c98
* net: move host_resolver files from net/base to net/dnsphajdan.jr@chromium.org2013-03-214-10/+10
| | | | | | | | BUG=70818 Review URL: https://codereview.chromium.org/12518036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189485 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Fix bug in session flow control which leaves streams stalledakalin@chromium.org2013-03-2110-140/+571
| | | | | | | | | | | | Use the term "send-stalled" in names consistently. Also remove more boilerplate from unit tests. BUG=178943 Review URL: https://codereview.chromium.org/12580010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189478 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up spdy_protocol* and spdy_test_utils.* files to match server versionakalin@chromium.org2013-03-203-142/+36
| | | | | | | | | | | | | | Remove unmaintained comments in spdy_protocol.h. Update link to SPDY/3 spec. Use base::HostToNetwork* instead of hton*. Add back accidentally-omitted SpdyProtocolDeathTest tests. Review URL: https://codereview.chromium.org/12782004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189372 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 189206 "Revert 189196"rtenneti@chromium.org2013-03-203-41/+182
| | | | | | | | | | | | | | | | | | | | | | > Revert 189196 > > net_unittests has been timing out spuriously around the time this was checked-in, rolling-back to see if it is related (see http://crbug.com/222298 for details). > > > SPDY - Fix for crash bug with CloseIdleConnections invalidating the iterator. > > > > BUG=181400 > > R=rch > > TEST=net unittests > > > > Review URL: https://chromiumcodereview.appspot.com/12918014 > > TBR=rtenneti@chromium.org > Review URL: https://codereview.chromium.org/12440056 TBR=eroman@chromium.org Review URL: https://codereview.chromium.org/12500011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189340 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 189196eroman@chromium.org2013-03-203-182/+41
| | | | | | | | | | | | | | | | | net_unittests has been timing out spuriously around the time this was checked-in, rolling-back to see if it is related (see http://crbug.com/222298 for details). > SPDY - Fix for crash bug with CloseIdleConnections invalidating the iterator. > > BUG=181400 > R=rch > TEST=net unittests > > Review URL: https://chromiumcodereview.appspot.com/12918014 TBR=rtenneti@chromium.org Review URL: https://codereview.chromium.org/12440056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189206 0039d316-1c4b-4281-b951-d872f2087c98
* SPDY - Fix for crash bug with CloseIdleConnections invalidating the iterator.rtenneti@chromium.org2013-03-203-41/+182
| | | | | | | | | | BUG=181400 R=rch TEST=net unittests Review URL: https://chromiumcodereview.appspot.com/12918014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189196 0039d316-1c4b-4281-b951-d872f2087c98
* [Net] Separate out priority field from HttpRequestInfoakalin@chromium.org2013-03-196-238/+270
| | | | | | | | | | | | | | This is in preparation for supporting reprioritization; HttpRequestInfo is assumed to be an unchanging struct, so priority can't live in it if it is something that can change. Also add DEFAULT_PRIORITY value and replace some uses of LOWEST with it. BUG=166689 Review URL: https://codereview.chromium.org/12833008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189099 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the unused HttpStreamBase::IsMoreDataBuffered().mmenke@chromium.org2013-03-192-5/+0
| | | | | | | | | | | | HttpNetworkTransaction used to use it in conjunction with Http proxy error checking, but now there's HttpProxyClientSocket which uses HttpStreamParser directly. BUG=none Review URL: https://chromiumcodereview.appspot.com/12538010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188886 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Refactor tests in preparation for a fix for a session flow control bugakalin@chromium.org2013-03-1815-668/+818
| | | | | | | | | | | Add tests for the session flow control bug. They will be enabled when the fix goes in. BUG=178943 Review URL: https://codereview.chromium.org/12743006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188878 0039d316-1c4b-4281-b951-d872f2087c98
* Add histograms to count when SpdySession GetPeer/LocalAddress return ↵rch@chromium.org2013-03-161-2/+9
| | | | | | | | | | | | ERR_SOCKET_NOT_CONNECTED. R=cbentzel@chromium.org BUG=179709 Review URL: https://chromiumcodereview.appspot.com/12898003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188601 0039d316-1c4b-4281-b951-d872f2087c98
* net: split net/ssl out of net/basephajdan.jr@chromium.org2013-03-1412-19/+19
| | | | | | | | | | | | | | | Also moves transport_security_state files to net/http. This change also updates the callers. R=wtc TBR=rsleevi,lambroslambrou,tony BUG=70818 Review URL: https://codereview.chromium.org/12680003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188119 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Refactor use of stream delegates in SpdySession testsakalin@chromium.org2013-03-144-174/+89
| | | | | | | | | | This is a follow-up to 187148; I missed a few delegates to audit. BUG=178943 Review URL: https://codereview.chromium.org/12623010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187992 0039d316-1c4b-4281-b951-d872f2087c98
* Add a ConnectionInfo enum which describes the kind of connection used to ↵rch@chromium.org2013-03-133-0/+25
| | | | | | | | fetch a response. This will eventually be made visible in window.chrome.loadTimings(). Review URL: https://chromiumcodereview.appspot.com/12518020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187763 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Bump up initial session receive window size to match streamsakalin@chromium.org2013-03-133-28/+63
| | | | | | | | | | | | We want the initial session receive window to be as large as the initial stream receive window. BUG=176592 Review URL: https://chromiumcodereview.appspot.com/12513009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187762 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Add SpdyStream unit tests for resuming after stall by flow controlakalin@chromium.org2013-03-135-469/+583
| | | | | | | | | | | | | These are a bit simpler than the similar SpdyNetworkTransaction tests. Also refactor test SpdyStream delegates and tests. BUG=176592 Review URL: https://chromiumcodereview.appspot.com/12727004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187723 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Audit delegates of SpdyStreamakalin@chromium.org2013-03-0910-69/+146
| | | | | | | | | | | | | | | | | | | | | | In particular, make sure that a SpdyStream's delegate is set immediately upon creation and that the delegates remove its reference to its SpdyStream on OnClose. This is in preparation for making SpdyStream non-ref-counted. Fix a few callbacks still using base::Unretained(). Fix a bug in SpdySession::CloseCreatedStream where the closed stream isn't notified it's being closed. TBR=rch@chromium.org BUG=178943 Review URL: https://chromiumcodereview.appspot.com/12436005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187148 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Rename SpdyStream::Write{Headers,StreamData} to Queue{...}akalin@chromium.org2013-03-086-71/+51
| | | | | | | | | | | | | | | | | | Those functions have ended up to always queue, so change the name to reflect that and remove the return value (which is always ERR_IO_PENDING). Remove some dead code handling the case when something other than ERR_IO_PENDING is returned. Make some functions take/return scoped_ptrs instead of raw pointers. BUG=176592 Review URL: https://codereview.chromium.org/12601004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187070 0039d316-1c4b-4281-b951-d872f2087c98
* [SPDY] Fix session flow control bug with incoming data for inactive streamsakalin@chromium.org2013-03-083-3/+41
| | | | | | | | | | | | The session receive flow control window should not decrease on incoming data for an inactive stream. BUG=176592 Review URL: https://chromiumcodereview.appspot.com/12648002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@186960 0039d316-1c4b-4281-b951-d872f2087c98
* Remove SpdyDataFlags from SpdyFramerVisitorInterface::OnStreamFrameData()akalin@chromium.org2013-03-0810-29/+28
| | | | | | | | | | This lands server change 43182358. TBR=rch@chromium.org Review URL: https://chromiumcodereview.appspot.com/12374038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@186874 0039d316-1c4b-4281-b951-d872f2087c98
* Cleaned up code to update listing of headersbengr@chromium.org2013-03-065-23/+29
| | | | | | | | | BUG=179382 Review URL: https://chromiumcodereview.appspot.com/12393058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@186357 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-07 02:25:24 +0000