summaryrefslogtreecommitdiffstats
path: root/net
Commit message (Collapse)AuthorAgeFilesLines
* Http cache: Add support for HEAD requests.rvargas@chromium.org2014-07-313-11/+382
| | | | | | | | | | | | | - A HEAD request is not cached by itself. - A HEAD request is returned from the cache if an entry is there. - A HEAD request updates the heders of a cached entry (after 304). - A HEAD request invalidates a cached entry (200 after revalidation). BUG=350407 Review URL: https://codereview.chromium.org/367973007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286647 0039d316-1c4b-4281-b951-d872f2087c98
* Disable SPDY and QUIC session pooling.rch@chromium.org2014-07-306-7/+23
| | | | | | | | | BUG=398925 R=agl@chromium.org Review URL: https://codereview.chromium.org/417013005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286598 0039d316-1c4b-4281-b951-d872f2087c98
* Start using L preview API for receiving proxy information. sgurun@chromium.org2014-07-301-4/+16
| | | | | | | | | | | BUG=398612 Android L preview API added a proxyinfo class for receiving proxy information. start using it. Review URL: https://codereview.chromium.org/426063006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286588 0039d316-1c4b-4281-b951-d872f2087c98
* Switched calculation of decode/holdback info to avoid confusion if the requestrdsmith@chromium.org2014-07-301-4/+5
| | | | | | | | | | | has gone away or the value was cached. BUG=None R=jar@chromium.org Review URL: https://codereview.chromium.org/413563003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286515 0039d316-1c4b-4281-b951-d872f2087c98
* Initial implementation of Chrome-Freshness header.ricea@chromium.org2014-07-305-23/+287
| | | | | | | | | | | | | | | | | | Add a header like Chrome-Freshness: max-age=30,stale-while-revalidate=60,age=10 when sending a revalidation request to a server which supplied the Cache-Control stale-while-revalidate directive on the previous response. Design doc: https://docs.google.com/document/d/1DMCIMAKjyKeYiu69jlI5OsO2pGyAMb81XflYK4hxsNM/edit BUG=348877 TEST=net_unittests Review URL: https://codereview.chromium.org/391763002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286482 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes to the SHA256 X509 Certificate fingerprint API.jww@chromium.org2014-07-303-26/+27
| | | | | | | | | | | In the original commit (285851), I missed several nits that wtc@ later pointed out. This commit addresses those corrections and suggestions. BUG=262615 Review URL: https://codereview.chromium.org/421243003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286477 0039d316-1c4b-4281-b951-d872f2087c98
* Implement JwkSerializer for OpenSSL.davidben@chromium.org2014-07-302-45/+109
| | | | | | | | | | | | | | | | Update the tests to use a real P-256 public key for the leading zero test. OpenSSL will refuse to import a public key that's not actually on the curve. (Actually importing it is somewhat tidier than sniffing at the SPKI and can handle other point formats.) This fixes ExternallyConnectableMessagingWithTlsChannelIdTest for the OpenSSL port. BUG=398662 Review URL: https://codereview.chromium.org/431453003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286454 0039d316-1c4b-4281-b951-d872f2087c98
* Implement ALPN for SSLClientSocketOpenSSL.davidben@chromium.org2014-07-3011-82/+83
| | | | | | | | | | | Also remove server_protos from the SSLClientSocket interface. NSS wasn't filling it in and NPN is going away eventually. With ALPN, this is less useful. BUG=388429 Review URL: https://codereview.chromium.org/423623002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286405 0039d316-1c4b-4281-b951-d872f2087c98
* Revert a change to the condition under which we will verify the proof inwtc@chromium.org2014-07-301-2/+11
| | | | | | | | | | | | | | | | | | STATE_INITIALIZE. We believe this change (in conjunction with the bug fixed in https://codereview.chromium.org/422623004/) caused the QUIC performance regression. Add comments to explain why the conditions for verifying the proof are different in STATE_INITIALIZE and STATE_RECV_REJ. R=jar@chromium.org,rch@chromium.org,rtenneti@chromium.org BUG=396185 Review URL: https://codereview.chromium.org/424213005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286400 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor net::MimeUtil media code to return "probably" for codecs the ↵acolwell@chromium.org2014-07-291-187/+358
| | | | | | | | | | | | | | | | | | | | | | | | | | platform likely supports. This patch amends the behaviour set for mp4 mime types Now browser returns "probably" for codecs which are confirmed to be shipped with Chrome. CodecPrameter| Before| Now -------------------------------- avc1.42E0xx -| maybe | probably avc1.4D40xx -| maybe | probably (non-Android) avc1.6400xx -| maybe | probably (non-Android) avc3.xxxxxx -| maybe | probably mp4a.6B -----| maybe | probably mp4a.69 -----| maybe | probably mp4a.67 -----| maybe | probably (non-Android) mp4a.40.2 ---| maybe | probably mp4a.40.5 ---| maybe | probably BUG=388317 TEST=MediaCanPlayTypeTest.* Review URL: https://codereview.chromium.org/422573005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286333 0039d316-1c4b-4281-b951-d872f2087c98
* Add a Clone method to ProofVerifyDetails to allow for the proof verifyrch@chromium.org2014-07-295-2/+32
| | | | | | | | | | | details to be copied when initiailizing a CachedState from a canonical entry. BUG=398465 Review URL: https://codereview.chromium.org/422623004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286305 0039d316-1c4b-4281-b951-d872f2087c98
* QUIC - track CHLO's reject reason for secure QUIC vs insecure QUIC.rtenneti@chromium.org2014-07-293-4/+12
| | | | | | | | R=rch@chromium.org, asvitkine@chromium.org Review URL: https://codereview.chromium.org/413363005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286150 0039d316-1c4b-4281-b951-d872f2087c98
* Implement TLS client auth in the OS X OpenSSL port.davidben@chromium.org2014-07-2913-50/+577
| | | | | | | | | | | | | | | | | | This introduces a openssl_platform_key.h that looks up and wraps a platform private key from the platform key store and returns an EVP_PKEY. It is implemented on Mac and left as a stub on Windows. This will be refactored with https://crbug.com/394131. The USE_OPENSSL_CERTS case has been left intact to preserve the existing tests on Linux but, possibly after the refactor, this will need to change as Linux and CrOS will likely still use OpenSSL handles for X509Certificate but will not likely want the OpenSSLClientKeyStore hack. BUG=394131 Review URL: https://codereview.chromium.org/396803002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286112 0039d316-1c4b-4281-b951-d872f2087c98
* Map WebSocket URL schemes to HTTP URL schemes for auth purposes.ricea@chromium.org2014-07-2910-39/+355
| | | | | | | | | | | | | | | | | | | This permits WebSocket connections to inherit credentials from HTTP pages, and matches the behaviour of other browsers. Design doc: https://docs.google.com/a/chromium.org/document/d/129rLtf5x3hvhP5rayLiSxnEjOXS8Z7EnLJgBL4CdwjI/edit Also consider any 401 or 407 results that reach the WebSocketStream URLRequest::Delegate to be unrecoverable errors. Also ensure that the response headers are reported back to the renderer when the developer tools are open and a 401 error happens. BUG=123862 Review URL: https://codereview.chromium.org/336263005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286108 0039d316-1c4b-4281-b951-d872f2087c98
* Move the test data that's used by net_unittests to net\data to simplify the ↵jam@chromium.org2014-07-296-7/+8
| | | | | | | | | | | | | | isolate files. The background is that url_fetcher was moved from chrome to content to net, but its test data didn't move along. R=davidben@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=286024 Review URL: https://codereview.chromium.org/420313003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286094 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 286024 "Move the test data that's used by net_unittests t..."mpearson@chromium.org2014-07-296-8/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Likely cause of failure on tree: http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/32682 net_unittests net_unittests net_unittests 127 disabled 6 flaky failed 5 ( 4 mins, 19 secs ) stdio Basic LargeGet LargeGetToTempFile ServerUnavailable ServerUnavailablePropagateResponse e.g., [ RUN ] URLFetcherDownloadProgressTest.Basic HTTP server started on http://127.0.0.1:54345... sending server_data: {"host": "127.0.0.1", "port": 54345} (36 bytes) File not found animate1.gif full path:/mnt/data/b/build/slave/Linux_Tests__dbg__1_/build/src/chrome/test/data/animate1.gif localhost - - [28/Jul/2014 16:51:57] code 404, message Not Found ../../net/url_request/url_fetcher_impl_unittest.cc:601: Failure Expected: (total) >= (progress), actual: -1 vs 190 ../../net/url_request/url_fetcher_impl_unittest.cc:603: Failure Value of: total Actual: -1 Expected: expected_total_ Which is: 19021 ../../net/url_request/url_fetcher_impl_unittest.cc:170: Failure Value of: source->GetResponseCode() Actual: 404 Expected: 200 [ FAILED ] URLFetcherDownloadProgressTest.Basic (205 ms) > Move the test data that's used by net_unittests to net\data to simplify the isolate files. > > The background is that url_fetcher was moved from chrome to content to net, but its test data didn't move along. > > R=davidben@chromium.org > > Review URL: https://codereview.chromium.org/420313003 TBR=jam@chromium.org Review URL: https://codereview.chromium.org/422063008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286036 0039d316-1c4b-4281-b951-d872f2087c98
* Move the test data that's used by net_unittests to net\data to simplify the ↵jam@chromium.org2014-07-286-7/+8
| | | | | | | | | | | | isolate files. The background is that url_fetcher was moved from chrome to content to net, but its test data didn't move along. R=davidben@chromium.org Review URL: https://codereview.chromium.org/420313003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286024 0039d316-1c4b-4281-b951-d872f2087c98
* Remove SPDY_PROXY_AUTH_ORIGIN from net/proxy/proxy_list.cc.kundaji@chromium.org2014-07-281-14/+7
| | | | | | | | | | | Pass in bypass_duration explicitly when updating proxy bypass duration instead of generating random duration for data reduction use case. BUG=396786 Review URL: https://codereview.chromium.org/415743002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285976 0039d316-1c4b-4281-b951-d872f2087c98
* net: several HSTS preloaded additions.agl@chromium.org2014-07-282-0/+14
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285954 0039d316-1c4b-4281-b951-d872f2087c98
* net: add (*.)domains.google.com to HSTS preloaded.agl@chromium.org2014-07-282-0/+2
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285946 0039d316-1c4b-4281-b951-d872f2087c98
* Ignore Set-Cookie Directive where both name and value are emptyxunjieli@chromium.org2014-07-283-3/+26
| | | | | | | | BUG=392295 Review URL: https://codereview.chromium.org/405233004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285932 0039d316-1c4b-4281-b951-d872f2087c98
* [WebSocket] Send a close frame when the renderer process is gone.yhirano@chromium.org2014-07-288-30/+90
| | | | | | | | | | | | | | Renderer processes may die without clearing its |WebSocketBridge|s. It is helpful to send a close frame with code = 1001 in such a case. BUG=392534 R=ricea@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285557 Review URL: https://codereview.chromium.org/390773002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285884 0039d316-1c4b-4281-b951-d872f2087c98
* Extract ScopedTestNSSDB from nss_util.pneubeck@chromium.org2014-07-287-13/+14
| | | | | | | | | | | | | | | | | | | | | Before ScopedTestNSSDB affected several slot getters from nss_util.h . This change reduces ScopedTestNSSDB to solely setup a temporary test DB and not influencing the global state in nss_util anymore. As a replacement for some of its old behavior, a new ScopedTestSystemNSSKeySlot is added, which allows to override the slot returned by GetSystemNSSKeySlot(). With this change it's now possible to write tests that need both a user and system NSS DB by using ScopedTestSystemNSSKeySlot. As a side-effect, GetPersistentNSSKeySlot() is now compiled on !OS_CHROMEOS only. BUG=210525 (For include changes:) R=rsleevi@chromium.org TBR=nkostylev@chromium.org, stevenjb@chromium.org Review URL: https://codereview.chromium.org/401623006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285881 0039d316-1c4b-4281-b951-d872f2087c98
* Add SHA-256 fingerprint functions to x509 certs.jww@chromium.org2014-07-283-0/+131
| | | | | | | | | | | | | | | Previously, the x509 cert implementation only had SHA-1 based fingerprinting for quickly creating indexing appropriate hashes. Since SHA-1 is likely cryptographically broken, these fingerprints should not be used for any security decisions. This adds slow SHA-256 fingerprint support for when fingerprints are needed for security decisions. R=rsleevi@chromium.org BUG=262615 Review URL: https://codereview.chromium.org/412263003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285851 0039d316-1c4b-4281-b951-d872f2087c98
* QUIC - Track the reason for sending InchoateClientHello in UMArtenneti@chromium.org2014-07-261-1/+34
| | | | | | | | | | | | | histogram. Possible reasons are: no/empty server config, expired server config, invalid server config and corrupted server config. R=rch@chromium.org, asvitkine@chromium.org Review URL: https://codereview.chromium.org/416983005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285765 0039d316-1c4b-4281-b951-d872f2087c98
* Implement TLS_FALLBACK_SCSV for SSLClientSocketOpenSSL.davidben@chromium.org2014-07-263-8/+5
| | | | | | | | | | | | | | In doing so, fix a bug in tlslite's TLS_FALLBACK_SCSV support; the fallback alert should be sent with the client's version. Otherwise OpenSSL reports SSL_R_UNSUPPORTED_PROTOCOL and doesn't report the alert. This behavior is probably not wrong as, if the server responds with a TLS version higher than what is supported, we can't really be sure of the parse. BUG=388425 Review URL: https://codereview.chromium.org/418553003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285764 0039d316-1c4b-4281-b951-d872f2087c98
* Update masters after the tryserver split.sergeyberezin@chromium.org2014-07-261-3/+7
| | | | | | | | | | | Also replace deprecated linux_chromium_rel with linux_chromium_rel_swarming, while I'm at it. R=agable@chromium.org, maruel@chromium.org BUG=395196 Review URL: https://codereview.chromium.org/415323002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285752 0039d316-1c4b-4281-b951-d872f2087c98
* Log the certificate subjects from the server certificate sent via QUIC.rch@chromium.org2014-07-254-0/+35
| | | | | | | | Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285446 Review URL: https://codereview.chromium.org/418723002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285721 0039d316-1c4b-4281-b951-d872f2087c98
* ParseHosts: Allow commas as separators on Mac OS Xttuttle@chromium.org2014-07-253-29/+151
| | | | | | | | | | | | Apparently, OS X allows commas as separators between hostnames in the hosts file. Treat commas the same as whitespace to support this. (Hostnames and IP addresses will never contain commas, so this shouldn't break anything.) BUG=396309 Review URL: https://codereview.chromium.org/415153002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285718 0039d316-1c4b-4281-b951-d872f2087c98
* Revert of [WebSocket] Send a close frame when the renderer process is gone. ↵kaznacheev@chromium.org2014-07-258-90/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/390773002/) Reason for revert: http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%282%29/builds/5350 Original issue's description: > [WebSocket] Send a close frame when the renderer process is gone. > > Renderer processes may die without clearing its |WebSocketBridge|s. > It is helpful to send a close frame with code = 1001 in such a case. > > BUG=392534 > R=ricea@chromium.org > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285557 TBR=ricea@chromium.org,jgraettinger@chromium.org,eroman@chromium.org,mmenke@chromium.org,yhirano@chromium.org NOTREECHECKS=true NOTRY=true BUG=392534 Review URL: https://codereview.chromium.org/416333004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285564 0039d316-1c4b-4281-b951-d872f2087c98
* [WebSocket] Send a close frame when the renderer process is gone.yhirano@chromium.org2014-07-258-30/+90
| | | | | | | | | | | | Renderer processes may die without clearing its |WebSocketBridge|s. It is helpful to send a close frame with code = 1001 in such a case. BUG=392534 R=ricea@chromium.org Review URL: https://codereview.chromium.org/390773002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285557 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the deprecated NSSCertDatabase::GetInstance() .pneubeck@chromium.org2014-07-255-114/+76
| | | | | | | | | | The NSSCertDatabase singleton for Linux is now maintained by nss_context_linux.cc . BUG=329735 Review URL: https://codereview.chromium.org/405973003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285551 0039d316-1c4b-4281-b951-d872f2087c98
* Revert of Reenable UpdateDynamicPKPMaxAge0 test. ↵jam@chromium.org2014-07-251-1/+8
| | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/293253002/) Reason for revert: this is still flaking. more info in bug TBR=palmer NOTRY=true Original issue's description: > Reenable UpdateDynamicPKPMaxAge0 test. > > BUG=375538 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=281863 Review URL: https://codereview.chromium.org/412353006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285508 0039d316-1c4b-4281-b951-d872f2087c98
* Make Pepper WebSocket UtilityGetProtocol test less flaky by making the wsh ↵tyoshino@chromium.org2014-07-251-0/+2
| | | | | | | | | | | | | | wait for close Attempt to fix the flakiness by making sure the server handler doesn't exit before the client closes. BUG=389084 R=jgraettinger,yhirano Review URL: https://codereview.chromium.org/410383003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285500 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to using SSL_get0_certificate_types for client auth.davidben@chromium.org2014-07-251-3/+2
| | | | | | | | | | | Upstream added SSL_get0_certificate_types while we added SSL_get_client_certificate_types in parallel. Switch to upstream's API. BUG=388000 Review URL: https://codereview.chromium.org/415853002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285498 0039d316-1c4b-4281-b951-d872f2087c98
* Update net/data/websocket/close-code-and-reason_wsh.pytyoshino@chromium.org2014-07-251-0/+3
| | | | | | | | | | | | Sync with one in the LayoutTests directory to include the code to wait for a close frame from the client. BUG=396399 R=jgraettinger,ricea Review URL: https://codereview.chromium.org/415953003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285497 0039d316-1c4b-4281-b951-d872f2087c98
* Remove Create*Value (except CreateNullValue)estade@chromium.org2014-07-251-4/+2
| | | | | | | | BUG=160586 Review URL: https://codereview.chromium.org/415423002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285496 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 285446 "Log the certificate subjects from the server cert..."hajimehoshi@chromium.org2014-07-254-35/+0
| | | | | | | | | | | | | | Tests failed on XP: http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/32148 > Log the certificate subjects from the server certificate sent via QUIC. > > Review URL: https://codereview.chromium.org/418723002 TBR=rch@chromium.org Review URL: https://codereview.chromium.org/415333005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285480 0039d316-1c4b-4281-b951-d872f2087c98
* Make the default alternate protcol probability threshold 1, not 0.rch@chromium.org2014-07-253-3/+10
| | | | | | | | | | | | | This means that by default, Chrome will honor headers like: Alternate-Protocol: 80:quic Alternate-Protocol: 80:quic,p=1 But not: Alternate-Protocol: 80:quic,p=.5 Eventually we need to switch to a dynamic threshold, but on step at a time. Review URL: https://codereview.chromium.org/411423003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285473 0039d316-1c4b-4281-b951-d872f2087c98
* Log the certificate subjects from the server certificate sent via QUIC.rch@chromium.org2014-07-254-0/+35
| | | | | | Review URL: https://codereview.chromium.org/418723002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285446 0039d316-1c4b-4281-b951-d872f2087c98
* Remove LOG(INFO)'s from spdy_network_transaction_unittest.cc - Revert r244946rch@chromium.org2014-07-251-14/+0
| | | | | | | | BUG=395354 Review URL: https://codereview.chromium.org/416153002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285444 0039d316-1c4b-4281-b951-d872f2087c98
* Rename ServerBoundCert => ChannelID to reflect the current namerch@chromium.org2014-07-2554-1761/+1754
| | | | | | | | | | of this feature. TBR=darin Review URL: https://codereview.chromium.org/356713005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285432 0039d316-1c4b-4281-b951-d872f2087c98
* Explain the three values written by WriteSecBuf, in particular, why thewtc@chromium.org2014-07-241-0/+4
| | | | | | | | | | | first two values are the same. R=rsleevi@chromium.org, glider@chromium.org BUG=none Review URL: https://codereview.chromium.org/418533002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285407 0039d316-1c4b-4281-b951-d872f2087c98
* net: add a couple of sites to HSTS preloaded.agl@chromium.org2014-07-242-1/+7
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285368 0039d316-1c4b-4281-b951-d872f2087c98
* Fix copy-pasto in MockSSPILibrary::QuerySecurityPackageInfo and ↵glider@chromium.org2014-07-241-2/+2
| | | | | | | | | | | MockSSPILibrary::DeleteSecurityContext BUG=NONE R=cbentzel@chromium.org Review URL: https://codereview.chromium.org/414443011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285191 0039d316-1c4b-4281-b951-d872f2087c98
* Fix WebSocket race between close and connect.ricea@chromium.org2014-07-242-0/+29
| | | | | | | | | | | | | | | | | | | If the WebSocket handshake was cancelled immediately after the connect succeeded before the ClientSocketHandle was fully initialised, then the socket would not be released back to the pool, and as a result the endpoint would not be unlocked. Fix by actively reclaiming the socket in WebSocketTransportClientSocketPool::CancelRequest(). Also add a test for this condition. BUG=394268, 389084 TEST=net_unittests Review URL: https://codereview.chromium.org/394113003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285175 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor unix domain socket.byungchul@chromium.org2014-07-2414-380/+1331
| | | | | | | | | | | | | | | | This is a pre-requisite for http server refactoring, https://codereview.chromium.org/296053012/. 1) Define UnixDomainClientSocket and UnixDomainServerSocket utilizing SocketLibevent. 2) Rename UnixDomainSocket to UnixDomainListenSocket to reduce confusion. 3) unittests BUG=371906 Review URL: https://codereview.chromium.org/376323002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285148 0039d316-1c4b-4281-b951-d872f2087c98
* Remove now unnecessary const_casts.davidben@chromium.org2014-07-241-1/+1
| | | | | | | | | | BoringSSL fixes this API to be const-correct. BUG=none Review URL: https://codereview.chromium.org/417683003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285134 0039d316-1c4b-4281-b951-d872f2087c98
* clang/win: Fix most -Wwriteable-strings warnings.thakis@chromium.org2014-07-233-7/+20
| | | | | | | | | | | | | | | | | | | | | | Many win32 APIs take non-const string pointers. I checked that MSDN documents them as _In_ and says that they are inputs, and then added const_cast<>s at the calling sites. (In one test, I introduced a helper struct so that there can be fewer casts.) This wasn't just busywork, I found one function that we were handing string literals where the documentation explicitly said that that's not valid (filed http://crbug.com/396705). I didn't change the DECLARE_REGISTRY_APPID_RESOURCEID() call in cloud_print; it sounds like that'll fix itself when we update to the 2014 sdk: http://connect.microsoft.com/VisualStudio/feedback/details/806376/atl-hindrances-to-adopting-new-strictstrings-conformance-option-in-vs2013 BUG=396705,82385 R=rnk@chromium.org, rsleevi@chromium.org, sergeyu@chromium.org, vitalybuka@chromium.org Review URL: https://codereview.chromium.org/413763003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285051 0039d316-1c4b-4281-b951-d872f2087c98
* Disable all the tests that are flaking more than 5% on Windows builders.jam@chromium.org2014-07-231-2/+14
| | | | | | | | | | | | Data from http://chromium-build-logs.appspot.com/flakiness_dashboard?master=chromium.win. BUG=386891,387124,396373,396384,396386,396387,396388,396390,396392,387124,396393,396395,396399,396402,396403,396404,396405,393149,396409,95557,396413,396414,375894 R=viettrungluu@chromium.org TBR=brettw, rvargas, sky Review URL: https://codereview.chromium.org/410843003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284977 0039d316-1c4b-4281-b951-d872f2087c98