| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
permit connection pooling of hosts with the same etld when TLS channel ID is used.
* Add tests for SpdySession::VerifyDomainAuthentication
* Add a new certificate for testing SpdySession::VerifyDomainAuthentication
BUG=
Review URL: https://chromiumcodereview.appspot.com/10916275
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156461 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:33872
Review URL: https://chromiumcodereview.appspot.com/10916094
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155720 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
R=eroman@chromium.org, yurys@chromium.org
BUG=73945
Review URL: https://chromiumcodereview.appspot.com/10870080
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155413 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/10907048
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154931 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=139700
Review URL: https://chromiumcodereview.appspot.com/10875017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154886 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
ResetOffset(), GetContentLength() and IsInMemory() are moved from UploadData to UploadDataStream.
This is an intermediate step to make simlar change to UploadElement
BUG=72001
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10878082
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154594 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
HttpTransaction::GetUploadProgress()
Add net::UploadProgress
Change return type of GetUploadSize() from uint64 to UploadProgress.
BUG=112607
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10834178
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154063 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
This reverts commit r153697: it was only landed on trunk for testing.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153883 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
This change is being landed temporarily on trunk for testing. It'll be
reverted once the change has be shown to be safe for merging.
BUG=139744
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153697 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=143965
TEST=network unittests. In about:histograms, we would see Net.SpdyVersion histogram
R=jar
Review URL: https://chromiumcodereview.appspot.com/10860073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152855 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
CID=104690,104691
CID_COUNT=2
R=jar@chromium.org
BUG=none
Review URL: https://chromiumcodereview.appspot.com/10857049
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152515 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To prepare for new framing implementation and tests.
- update test data to mimic real cases and follow the latest spec
- reorganize utility functions for tests
BUG=42320
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10832335
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152484 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=115047
R=willchan@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10825437
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152296 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These tests fail on Linux redux, that uses OpenSSL, due to
ec_private_key_openssl.cc which is not implemented yet.
They were already disabled on Android through an entry in
build/android/gtest_filter/net_unittests_disabled.
TEST=net_unittests
BUG=142833,140991
Review URL: https://chromiumcodereview.appspot.com/10830354
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152163 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(This is a reland of r151502, which was reverted in r151517 because it
broke the Linux Official build.)
This may break the Official build for a brief window while a two-sided
patch lands.
BUG=139744
Review URL: https://chromiumcodereview.appspot.com/10837057
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151720 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Introduce a completion waiting list, and check which kind of frame is sent.
OnDataSent() is invoked iff the frame wis data frame.
BUG=42320
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10828129
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151647 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
This reverts r151502.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151517 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=139744
Review URL: https://chromiumcodereview.appspot.com/10837057
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151502 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Due to a bug, the Clang-plugin style checker failed to warn about
inline constructors, destructors, non-empty virtual methods, etc.
for nested classes.
The plugin has been fixed, and this patch is part of a clean-up of all
the code that now causes the plugin to issue errors.
BUG=139346
Review URL: https://chromiumcodereview.appspot.com/10854063
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150917 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Don't trust most non-success responses to a CONNECT request -- as the BUG=
explains, the rest of the stack will treat the response as if it came from
the target server, not the proxy. This trivially lets a proxy run code as
any HTTPS site the user tries to connect to, which is Very Bad™.
Do, however, accept 302 responses, but sanitize them so they contain only
the Location header and no response body. Many proxies use this for login
pages, so we can't break it.
Update the HttpProxyClientSocketPool unittests to expect failure in all but
the 302 case, and add a 302-specific test case.
BUG=137891
TEST=Added cases to Http- and SpdyProxyClientSocket unittests. net_unittests pass.
Review URL: https://chromiumcodereview.appspot.com/10825030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150749 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If we receive a data frame without a status or without
a version header, we close the stream with a PROTOCOL ERROR.
Small bug fix to HttpNetworkTransaction to access the
ResponseHeaders only if headers are there.
In SpdyStream, retrun a SPDY_PROTOCOL_ERROR if we have pending
data frames, but we haven't received "status" and "version"
headers. (rch: removed the DCHECK for unit tests).
SpdyHttpStream's OnDataReceived used to expect that it would
be called only when all required headers are received. Converted
the DCHECK into an error condition and SpdyStream closes the
stream with PROTOCOL ERROR if OnDataReceived returns a network
error.
BUG=135485
R=rch@chromium.org
TEST=network unittests
Review URL: https://chromiumcodereview.appspot.com/10836084
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150121 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=139700
Review URL: https://chromiumcodereview.appspot.com/10807088
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150112 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To support WebSocket over SPDY, SpdyStream should provide a interface to
send HEADERS control frames for SpdyWebSocketStream.
This is because WebSocket over SPDY requires to convert a WebSocket frame
into one HEADERS frame and subsequent DATA frame(s) in the spec.
See also http://goo.gl/mJCrx the spec draft 9, "Frame mapping" section.
BUG=42320
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10810069
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149808 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
CID_COUNT=2
CID=104562,104563
BUG=none
R=khorimoto@chromium.org
TBR=rch
Review URL: https://chromiumcodereview.appspot.com/10828138
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149778 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=135203
TEST=
Review URL: https://chromiumcodereview.appspot.com/10824115
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149482 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=139700
Review URL: https://chromiumcodereview.appspot.com/10832098
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149461 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
underlying map had changed, making the iterator invalid.
BUG=139518
TEST=SpdySessionSpdy*Test.CloseSessionWithTwoCreatedStreams
Review URL: https://chromiumcodereview.appspot.com/10821086
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149071 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
considered active by the spdy session.
BUG=139375
TEST=SpdySessionSpdy2Test.CancelStream
Review URL: https://chromiumcodereview.appspot.com/10829066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148865 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, MockRead and MockWrite are defined as the same type by using
typedef. However, these types are different types and should not be used
interchangably.
This change gives MockRead and MockWrite distinct types by using templates
so the compiler can cause an error when these types are used interchangably.
Some misuses of MockRead and MockWrite were found, so these are fixed in this
change, too.
BUG=none
R=mmenke@chromium.org,akalin@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10796085
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148782 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
produce data. This allows us to lazily allocate a stream id.
The second CL was reverted because of use-after-free problems. Producers were deleted before they were pop()'d from the WriteQueue, which turns out to be a no-no. This version fixes this defect.
The initial CL was reverted because of memory leaks. Both SpdyIOBufferProducers leaked the SpdyFrame they owned. The second version of the CL fixes this defect.
Attempting to re-land 144649
Revert 147692 - Revert 144655 - Revert 144649
BUG=111708
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10815074
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148660 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- When a client receives a SYN_STREAM from the server with
an Associated-To-Stream-ID of 0, it replies with a
RST_STREAM with error code REFUSED_STREAM
BUG=133995, 133247
R=willchan
TEST=network unit tests
Review URL: https://chromiumcodereview.appspot.com/10816026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148115 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
and writes start at zero and are continuous.
Review URL: https://chromiumcodereview.appspot.com/10795012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147749 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
streams that are ready to produce data. This allows us to lazily allocate a stream id.
This patch still brakes net_unittests
The CL was reverted because of memory leaks. Both SpdyIOBufferProducers leaked the SpdyFrame they owned. This version of the CL fixes this defect.
Attempting to re-land 144649
Revert 144655 - Revert 144649
BUG=111708
TEST=
Review URL: https://chromiumcodereview.appspot.com/10803041
TBR=rch@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10807057
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147706 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ready to produce data. This allows us to lazily allocate a stream id.
The CL was reverted because of memory leaks. Both SpdyIOBufferProducers leaked the SpdyFrame they owned. This version of the CL fixes this defect.
Attempting to re-land 144649
Revert 144655 - Revert 144649
BUG=111708
TEST=
Review URL: https://chromiumcodereview.appspot.com/10803041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147692 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
This anti-pattern simply complicates tests and adds no value.
Review URL: https://chromiumcodereview.appspot.com/10809022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147567 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=113424
TEST=SpdySessionSpdy[23]Test.DeleteExpiredPushStreams
Review URL: https://chromiumcodereview.appspot.com/10790064
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147505 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
even if the session is not "secure". This required refactoring StreamSocket
to add WasNpnNegotiated() and GetSSLInfo() methods.
This allows for a change to SpdySession::GetSSLInfo to accurately return the correct SSLInfo in the case of SPDY Proxy sessions.
BUG=134690
TEST=\*DoNotUseSpdySessionIfCertDoesNotMatch\*
Review URL: https://chromiumcodereview.appspot.com/10690122
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147479 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=
TEST=net_unittests SpdyStreamSpdy2Test SpdyStreamSpdy3Test
Review URL: https://chromiumcodereview.appspot.com/10802006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147458 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=113424
TEST=SpdySessionSpdy[2,3]Test.DeleteExpiredPushStreams
Review URL: https://chromiumcodereview.appspot.com/10702189
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147321 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/10784055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147257 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We could not privide correct flags information for SPDY_SESSION_RECV_DATA.
This is because SPDY_SESSION_RECV_DATA events are not mapped into actual SPDY
data frames.
This change removes the flags information from SPDY_SESSION_RECV_DATA and adds
a boolean fin flag instead.
BUG=79153
TEST=manual check on chrome://net-internals/#events SPDY_SESSION
Review URL: https://chromiumcodereview.appspot.com/10736007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146367 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
in SendBody instead of SendBodyComplete.
SpdyHttpStream inherits from ChunkCallback (SpdyStream doesn't inherit
from ChunkCallback). OnChunkAvailable is handled by SpdyHttpStream.
In unit tests, added new tests (similar to http_stream_parser_unittest
for chunked uploads), to delay sending of chunked data.
Also added sending of window_update during chunk uploads. Verified
that these tests fail with the old code and run ok with the new changes.
This change also fixed chunked uploads hanging (or stalling) for
speech recgonition.
Many many thanks to sleevi for suggesting the way to clean up
the SpdyHttpStream's SendBody and SendBodyComplete code.
R=rch@chromium.org, rsleevi@chromium.org
TEST=network unit tests
BUG=113107, 136044
Review URL: https://chromiumcodereview.appspot.com/10689034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146255 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
For context see this thread:
https://groups.google.com/a/chromium.org/forum/?fromgroups#!topic/chromium-dev/RMcVNGjB4II
TBR=thakis,pkasting,jam
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146163 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/10763011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146062 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=123295
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10690104
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145845 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
SpdyFramerVisitorInterface::OnHeaders() and OnSynStream().
Remove SpdyFramerVisitorInterface::OnControl().
Lands server change 31251012, 31269046, 31356969.
Review URL: https://chromiumcodereview.appspot.com/10690066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145763 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
streams that are ready to produce data. This allows us to lazily allocate a stream id.
BUG=111708
Review URL: https://chromiumcodereview.appspot.com/10448083
TBR=rch@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10710008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144655 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
ready to produce data. This allows us to lazily allocate a stream id.
BUG=111708
Review URL: https://chromiumcodereview.appspot.com/10448083
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144649 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=134864
TEST=Visit an HTTP URL through a SPDY proxy. View the session in chrome://net-internals/#spdy and notice that the protocol is not "unknow".
Review URL: https://chromiumcodereview.appspot.com/10695010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144532 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
SpdySession::kMaxPushedConcurrentStreams
Add comment to SpdySession global variables:
g_init_max_concurrent_streams = 100;
g_max_concurrent_stream_limit = 256;
Fix bug in SpdySession::ResetStaticSettingsToInit introduced in r142228.
Review URL: https://chromiumcodereview.appspot.com/10676003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144307 0039d316-1c4b-4281-b951-d872f2087c98
|