| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
BUG=109310
Review URL: http://codereview.chromium.org/9139070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117709 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
canceled/completed/killed.
With this CL, URLRequestJobs notify their NetworkDelegate when they are being destroyed to insure that the NetworkDelegate does not access any of their variables or call callbacks.
Also the WebRequest API does not execute any callbacks if they URLRequest has informed the API that the Request has been completed or canceled.
Both cases could happen in rare race conditions.
BUG=108968
TEST=no
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=117342
Review URL: http://codereview.chromium.org/9147018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117685 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change broke WebKit test http/tests/xmlhttprequest/methods-lower-case.html
This reverts commit a964df83606a31b3674e20989e7dc6db9f3d4e86.
TBR=cbentzel@chromium.org
Review URL: http://codereview.chromium.org/9207003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117641 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This wasn't working because ResourceDispatcherHost::BeginDownload was overriding the method. I traced through all callers of BeginDownload and confirmed that the method did not need to be set.
BUG=110044
TEST=DownloadsApiTest.Downloads [the downloadPost test case]
Review URL: http://codereview.chromium.org/9148070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117637 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/9187060
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117635 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Turns out the test passes OK (given access to java side pieces, failure of which this bug masks)
BUG=109783
TEST=
Review URL: http://codereview.chromium.org/9187007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117632 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
canceled/completed/killed.
With this CL, URLRequestJobs notify their NetworkDelegate when they are being destroyed to insure that the NetworkDelegate does not access any of their variables or call callbacks.
Also the WebRequest API does not execute any callbacks if they URLRequest has informed the API that the Request has been completed or canceled.
Both cases could happen in rare race conditions.
BUG=108968
TEST=no
Review URL: http://codereview.chromium.org/9147018
TBR=battre@chromium.org
Review URL: http://codereview.chromium.org/9173005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117620 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is in preparation for reworking UploadDataStream.
We'll make more changes in the upcoming patches.
BUG=72001
TEST=try bots
Review URL: http://codereview.chromium.org/9187021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117588 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=try bots.
Review URL: http://codereview.chromium.org/9139041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117584 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
STATE_HANDSHAKE.
After transport I/O completes synchronously, stay in the handshake
loop only if the next state is STATE_HANDSHAKE, which is the only
state that requires transport I/O to make progress. If the next state
is any other state, we should ignore network_moved when determing if we
should stay in the handshake loop.
This changelist is the synchronous completion version of
http://codereview.chromium.org/660131, which dealt with this bug
when transport I/O completes asynchronously.
R=agl@chromium.org
BUG=109706
TEST=Run a Chrome debug build with --enable-origin-bound-certs and
connect to a Google server such as gmail over HTTPS. Should not hit
a DCHECK failure in net_log.cc.
Review URL: http://codereview.chromium.org/9172005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117578 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
SYN_STREAM/SYN_REPLY headers.
This also fixes bug when server sends SYN_REPLY in separate chunks
(i.e. header + zlib chunks ) chrome fails to recognize it.
BUG=108596
R=jar
TEST=ASAN tests and network unit tests
Review URL: http://codereview.chromium.org/9185036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117563 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117437 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
URLRequest::AppendFileToUpload() is no longer used
as now file uploads are done with UploadData.
TEST=chrome and tests build as before.
BUG=72001
Review URL: http://codereview.chromium.org/9148016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117357 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
canceled/completed/killed.
With this CL, URLRequestJobs notify their NetworkDelegate when they are being destroyed to insure that the NetworkDelegate does not access any of their variables or call callbacks.
Also the WebRequest API does not execute any callbacks if they URLRequest has informed the API that the Request has been completed or canceled.
Both cases could happen in rare race conditions.
BUG=108968
TEST=no
Review URL: http://codereview.chromium.org/9147018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117342 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117310 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
(There aren't any XP/Vista trybots any more.)
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117298 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
(We don't have trybots any longer for those platforms.)
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117265 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=net_unittests
Review URL: http://codereview.chromium.org/9152019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117242 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=net_unittests
Review URL: http://codereview.chromium.org/9153014
TBR=agl@chromium.org
Review URL: http://codereview.chromium.org/9186009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117236 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=net_unittests
Review URL: http://codereview.chromium.org/9153014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117225 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
It's #ifdef'ed USE_NSS for now because we don't yet have implementations for
Windows and OS X.
BUG=none
TEST=net_unittests
Review URL: http://codereview.chromium.org/9152011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117223 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The initialization function is split into two:
- to register for SCNetworkReachability notifications,
- to register for SCDynamicStore notifications.
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/9150002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117200 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Per wtc.
Note that we purposefully do not set MODE_STRICT when processing
Public-Key-Pins headers, because it is possible (and desirable by some) to
note and validate pins when using HTTPS, but not to require HTTPS. The
dynamic key pinning spec requires this behavior, although my first draft and
instincts were also to have pinning imply MODE_STRICT.
BUG=78369, 109772
TEST=net_unittests TransportSecurityStateTest.*, browser_tests NetInternalsTest.*
Review URL: http://codereview.chromium.org/9167024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117175 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change allows CRLSets to include a list of blocked SPKI fingerprints,
which may save us doing emergency binary pushes in the future.
It also corrects a bug where the NSS code was passing in the full SPKI rather
than the SHA256 hash.
BUG=none
TEST=net_unittests
Review URL: http://codereview.chromium.org/9149010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117069 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Improves code coverage.
TEST=./net_unittests --gtest_filter=HostResolverImplTest.FailedAsynchronousLookup
Review URL: http://codereview.chromium.org/9122013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116879 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=109621
TEST=none
Review URL: http://codereview.chromium.org/8956067
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116873 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Deleted all code associated with SPDY_INTERPRET_CONTROL_FRAME_COMMON_HEADER
- Deleted OldProcessControlFrameHeaderBlock and
OldIncrementallyDecompressControlFrameHeaderData
- Changed unittests to reflect the above changes. Deleted uses of ParseHeaderBlock
(server code doesn't use ParseHeaderBlock).
BUG=108608
R=wtc
TEST=network unit tests.
Review URL: http://codereview.chromium.org/9124022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116814 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Wasn't able to duplicate ASAN tests failing in
UnclosedStreamDataCompressors on my linux build.
Enabling it to see if the failure was flaky.
BUG=108596
R=rch
TEST=network unittests.
Review URL: http://codereview.chromium.org/8982009
TBR=rtenneti@chromium.org
Review URL: http://codereview.chromium.org/9141002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116777 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Wasn't able to duplicate ASAN tests failing in
UnclosedStreamDataCompressors on my linux build.
Enabling it to see if the failure was flaky.
BUG=108596
R=rch
TEST=network unittests.
Review URL: http://codereview.chromium.org/8982009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116772 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Over 341 CLs, in ~3 months, touching 3251 unique files!
Top 5 most CLs:
(121) jhawkins
( 45) dcheng
( 24) achuith
( 23) csilv
( 12) tfarina
( 12) groby
~1000 files touched:
(918) jhawkins
100+ files touched:
(486) ajwong
(385) willchan
(372) dcheng
(126) csilv
(123) fischman
(112) sergeyu
49+ files touched:
(65) tfarina
(57) acolwell
(52) adamk
(49) tzik
BUG=35223
TEST=existing
Review URL: http://codereview.chromium.org/9114020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116748 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This will be used in HostResolverImpl to dispatch Jobs.
BUG=109426
TEST=./net_unittests --gtest_filter=PriorityQueue*:PrioritizedDispatcher*
Review URL: http://codereview.chromium.org/9113022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116736 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The SSLConfig was a poor choice of location to carry the CRLSet because the
CRLSet can be updated while Chrome is running, but the SSLConfig is relatively
static and is cached in several places in the code.
This change causes the locations which call X509Certificate::Verify to grab a
new reference to the current CRLSet.
BUG=none
TEST=compiles
Review URL: http://codereview.chromium.org/9044011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116720 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently the use of a CRLSet replaces online revocation checking. With this
change, CRL sets become purely an additional check.
This allows us to enable CRL sets to test the full pipeline without changing
any other variables.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/9044012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116688 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
http://codereview.chromium.org/9041015/
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116684 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
sure that the number of entries on the file is not out of
sync.
* Check the counters on every session
* Make sure that num_entries is always >= number of used bits
(create/delete order)
* Make sure that the updating flag works for gcc & co.
* Fail initialization if we cannot fix the file.
BUG=108375
TEST=net_unittests
Review URL: http://codereview.chromium.org/9016033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116565 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now that we can serialise CRL sets, we can handle delta updates by applying
them to the current CRL set and writing the result back to disk.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/9041015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116533 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
An exponential poll delay is used which doubles the delay after each attempt (for instance polls might occur at 4 seconds, then 8 seconds, then 16 seconds, up to a maximum of 2 minutes).
This same mechanism also applies to the resolution of "auto-detect" settings.
The intent of this change is to make it possible to
(1) Notice server-side changes made to the PAC scripts
(2) Recover from spurious network errors when initially downloading PAC scripts.
BUG=90581
TEST=Configure Chrome to use a PAC script which fails to load (i.e. a non-existent file:// url, or remap the host's DNS record). Start Chrome and verify using about:net-internals that it has failed to fetch the PAC script and has fallen back to DIRECT settings. Fix the PAC URL to work and wait for chrome to notice the change and reconfigure itself (will take a maximum of 2 minutes).
Review URL: http://codereview.chromium.org/9078003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116462 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Upon receipt of the Public-Key-Pins header, check the syntax and the pins, and
set the pins in the TransportSecurityState. From then on, use these new
dynamic pins to validate TLS connections: as with preloaded pins, refuse to
connect to TLS servers that fail the pin check.
The Public-Key-Pins header is defined in an IETF Internet-Draft, and
discussed on the websec@ietf.org mailing list.
Clarified TransportSecurityState member function and field documentation.
Also: Minor "gcl lint" repairs, and a new typedef
std::vector<net::SHA1Fingerprint> FingerprintVector used everywhere relevant.
BUG=78369
TEST=net_unittests TransportSecurityStateTest.*, browser_tests NetInternalsTest.*
Review URL: http://codereview.chromium.org/8082016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116443 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=107479
TEST=net_unittests X509CertificateTest.*
Review URL: http://codereview.chromium.org/8956061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116442 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Originally, I tried to get rid of the DirectoryLister::Core class to simplify the implementation of DirectoryLister. What I didn't realize was that the purpose of the Core was to outlive the DirectoryLister itself in case the DirectoryLister is destroyed. I think this is probably the most straightforward: Keep the Core, but still make the implementation simpler than it was before we did the base::Bind conversion.
For reference, the previous CL was http://codereview.chromium.org/8989052
BUG=NONE
TEST=visit any file:// url
Review URL: http://codereview.chromium.org/9008059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116394 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
R=ncarter
BUG=b/3051801
TEST=it builds
Review URL: http://codereview.chromium.org/8966034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116319 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Resubmit of http://codereview.chromium.org/9026001/
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/8989052
TBR=tbreisacher@chromium.org
Review URL: http://codereview.chromium.org/9008058
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116268 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Resubmit of http://codereview.chromium.org/9026001/
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/8989052
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116259 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
http://codereview.chromium.org/9035012"
The hanging test is unrelated to this change (it's merely tickled by a
potential timing change).
BUG=50346
TEST=none
TBR=awong
Review URL: http://codereview.chromium.org/9088002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116252 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Kill off NewRunnableMethod() and remove silly declarations of unused ScopedRunnableMethodFactory(s).
BUG=none
TEST=existing.
Review URL: http://codereview.chromium.org/9034032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116250 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
R=groby,awong,csilv
Review URL: http://codereview.chromium.org/9028009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116169 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
R=groby,awong,csilv
Review URL: http://codereview.chromium.org/9064007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116158 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
The net unit tests have been red under windows tsan continuously since this landed.
BUG=50346
TEST=none
TBR=jhawkins
Review URL: http://codereview.chromium.org/9028008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116102 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
TBR=willchan,brettw,tony
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=115997
Review URL: http://codereview.chromium.org/9004051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116026 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
TBR=rtenneti@chromium.org
CID=102867
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/9034028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116010 0039d316-1c4b-4281-b951-d872f2087c98
|