| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add UploadFileElementReaderSync and use it from Chrome Frame.
Remove UploadDataStream::InitSync/ReadSync
Remove UploadElementReader::InitSync/ReadSync
Allow passing null callbacks for Init/Read when callers can be confident that the operation finishes synchronously.
BUG=None
TEST=net_unittests, chrome_frame_net_tests
TBR=ananta@chromium.org for chrome_frame/urlmon_upload_data_stream.cc
Review URL: https://chromiumcodereview.appspot.com/11587007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174133 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
able to write header blocks without control frames, since the information in syn streams should be handled at the QUIC layer.
Merge internal change: 33999615
Review URL: https://chromiumcodereview.appspot.com/11635025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174030 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=165264
Review URL: https://codereview.chromium.org/11632002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173801 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
that error.
BUG=164363
Review URL: https://codereview.chromium.org/11464028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173227 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change argument of UploadDataStream's ctor from UploadData to ScopedVector<UploadElementReader>
Move chunk related logic from UploadData to UploadDataStream.
Move UploadElementReader creation code from UploadElementReader::Create to URLRequest::set_upload. (this code will soon be moved again to src/webkit/glue/resource_request_body.cc)
BUG=156574
TEST=net_unittests
TBR=jam@chromium.org for chrome/browser/automation (already reviewed by ananta@)
Review URL: https://chromiumcodereview.appspot.com/11419220
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172295 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Simplifies the creation of HttpNetworkSession::Params in IOThread by creating a method to do just that. Fixes a bug in chrome/browser/profiles/profile_io_data.cc where some of the Globals were not being consulted when the HttpNetworkSession was constructed.
BUG=162571
Review URL: https://chromiumcodereview.appspot.com/11415219
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170967 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Use SpdySessionDependencies in HttpNetworkTransaction and
HttpProxyClientSocketPool tests, instead of hand-rolled alternatives.BUG=
Review URL: https://chromiumcodereview.appspot.com/11348333
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170687 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There are some code paths which calls SpdyHttpStream::GetUploadProgress() before the stream gets initialized.
Also fixing some CHECKs to make them securer.
BUG=161823
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/11316094
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168758 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Also add to base namespace.
BUG=
Review URL: https://codereview.chromium.org/11359217
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168281 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
I'll need that for my upcoming FTP CL.
BUG=131847
Review URL: https://codereview.chromium.org/11280014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168237 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
HttpInfo::upload_data is used instead.
BUG=156574
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/11361116
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167873 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
RunAllPending() is deprecated and we should switch to RunUntilIdle().
BUG=131220
TBR=willchan@chromium.org
Review URL: https://chromiumcodereview.appspot.com/11361263
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167705 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
URLRequestHttpJob
URLRequestHttpJob creates and owns UploadDataStream.
Change type of HttpRequestInfo::upload_data from UploadData* to UploadDataStream*.
Add UploadDataStream::identifier().
BUG=156574
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/11269039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167696 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
- move GetTestCertsDirectory() from net/base/certs_test_util
BUG=157531
TEST=build all
TBR=wtc@chromium.org
Review URL: https://chromiumcodereview.appspot.com/11342034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165348 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
to HttpStream
This change is a preparation step before moving ownership of UploadDataStream to URLRequest.
HttpStream::SendRequest takes UploadDataStream* instead of scoped_ptr<UploadDataStream>.
HttpNetworkTransaction does not release the ownership of UploadDataStream.
Some include cleanups.
BUG=155242
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/11269066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165337 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/11337031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165056 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
several.
BUG=none
TEST=none
Review URL: https://codereview.chromium.org/11268002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@164164 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
scoped_ptr.
Move HostResolver factory methods to host_resolver.cc.
This also fixes a double-free in ShellURLRequestContextGetter.
Review URL: https://chromiumcodereview.appspot.com/10831277
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163402 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The new asynchronous reading logic is also used for chunked upload.
UploadDataStream::ReadSync() uses the same code path as Read().
net::ChunkCallback is removed and base::Closure is used instead.
OnChunkAvailable() is added to UploadDataStream and removed from HttpStreamParser and SpdyHttpStream.
HttpStreamParser's request body reading logic got simplified.
BUG=152148
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/11187023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163226 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL disables both the workaround for http://crbug.com/139744
and SPDY compression when using system zlib.
Google Chrome should not be affected by this change (bundled
patched zlib still used, as well as compression).
This also fixes another uncovered problem with system minizip.
BUG=29048
TEST=none
Review URL: https://chromiumcodereview.appspot.com/11194068
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162911 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
ReadSync() is still used in some places, Chrome Frame, tests, HttpStreamParser code which always handles in memory data.
BUG=72001
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10978041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162670 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename existing Read() to ReadSync()
Change type of |buf| from char* to scoped_refptr<IOBuffer> so that async operation is always performed safely
Add an asynchronous implementation, Read()
The newly added Read() is not used yet, all users still use the old version ReadSync()
BUG=72001
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10910268
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162343 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
NOTRY=true
R=rsleevi@chromium.org
BUG=155147, 143646, 155110
Review URL: https://chromiumcodereview.appspot.com/11128003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@161781 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the response for the ping from the server, use the
same ping sequence number as server has sent us.
Added test cases for pings from server.
R=willchan
BUG=152701
TEST=network unit tests
Review URL: https://chromiumcodereview.appspot.com/10979079
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@160229 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, SPDY cookies would look like:
"FOO=BAR;BAR=BOO;WIBBLE=WOBBLE;"
With this patch they will look like:
"FOO=BAR; BAR=BOO; WIBBLE=WOBBLE"
This will hurt compression a little as cookies are only matched completely so:
Cookie: FOO=BAR
|-----|
won't match against the next Cookie header with an additional cookie:
Cookie: FOO=BAR; BAR=BOO
|------||------|
However, once the set of cookies is stable, it does fine.
BUG=151433
Review URL: https://chromiumcodereview.appspot.com/10957038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158088 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This changes ECSignatureCreator to use the hash function that SPDY
expects (SHA-256). There are no other users of ECSignatureCreator in
the tree so I'm going to defer making these choices parameters until there's
a benefit to be had.
It also adds DecodeSignature to convert from ASN.1 signatures to the `raw'
form that SPDY needs.
BUG=none
Review URL: https://chromiumcodereview.appspot.com/10910226
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157551 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename the existing synchronous version to InitSync() and add a new asynchronous version, Init().
However, the new asynchronous version is not used yet.
The existing code is changed to use the synchronous version.
BUG=72001
TEST=net_unittests
Review URL: https://chromiumcodereview.appspot.com/10913179
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157344 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|