diff options
author | Iain Merrick <husky@google.com> | 2010-10-05 16:38:29 +0100 |
---|---|---|
committer | Iain Merrick <husky@google.com> | 2010-10-19 14:37:55 +0100 |
commit | bc7e0823f37810f402bf7f115ee7ccd673f5ac34 (patch) | |
tree | 6a41ff955d13aba0bcc56b9ce429ab05d566501e /net/http/http_network_transaction.cc | |
parent | 3345a6884c488ff3a535c2c9acdd33d74b37e311 (diff) | |
download | external_chromium-bc7e0823f37810f402bf7f115ee7ccd673f5ac34.zip external_chromium-bc7e0823f37810f402bf7f115ee7ccd673f5ac34.tar.gz external_chromium-bc7e0823f37810f402bf7f115ee7ccd673f5ac34.tar.bz2 |
Merge Chromium at 7.0.540.0 : Resolve conflicts.
connection.cc, statement.cc: SQL headers moved one level deeper.
http://src.chromium.org/viewvc/chrome?view=rev&revision=56619
base_paths_posix.cc: EnvVarGetter renamed to Environment.
http://src.chromium.org/viewvc/chrome?view=rev&revision=55326
logging.cc: VLOG added, skipping in Android to minimize bloat.
http://src.chromium.org/viewvc/chrome?view=rev&revision=60976
process_util, spellcheck_host_observer, url_fetcher:
Virtual destructors added (duplicating existing Android change).
http://src.chromium.org/viewvc/chrome?view=rev&revision=62339
autofill_host.h: this is an Android-only file, but it depends on
AutoFillSuggestionsReturn in render_view_host.h. Icons added in
http://src.chromium.org/viewvc/chrome?view=rev&revision=54052
autofill_manager, personal_data_manager: InfoBarDelegate added.
http://src.chromium.org/viewvc/chrome?view=rev&revision=51908
credit_card.cc: #include <string> added for card type detection.
http://src.chromium.org/viewvc/chrome?view=rev&revision=54280
web_data_service.h: #includes reorganised to reduce build time.
http://src.chromium.org/viewvc/chrome?view=rev&revision=58080
host_resolver_impl.cc: parameter added to constructor.
http://src.chromium.org/viewvc/chrome?view=rev&revision=57189
host_resolver_proc.cc: retry logic updated.
http://src.chromium.org/viewvc/chrome?view=rev&revision=58534
http_network_transaction.cc: stream initialization refactored.
http://src.chromium.org/viewvc/chrome?view=rev&revision=57342
net_resources.target.mk: generated from net.gyp, using latest version.
proxy_service.cc: ProxyConfigService for Chrome, clashes with Android.
http://src.chromium.org/viewvc/chrome?view=rev&revision=57204
Change-Id: I0e05226051d67492692bdb1a5430b2f15ce72aa0
Diffstat (limited to 'net/http/http_network_transaction.cc')
-rw-r--r-- | net/http/http_network_transaction.cc | 69 |
1 files changed, 0 insertions, 69 deletions
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc index 0f0a7ea..3291710 100644 --- a/net/http/http_network_transaction.cc +++ b/net/http/http_network_transaction.cc @@ -1149,76 +1149,7 @@ void HttpNetworkTransaction::ResetConnectionAndRequestForResend() { // headers, but we may need to resend the CONNECT request first to recreate // the SSL tunnel. request_headers_.clear(); -<<<<<<< HEAD - next_state_ = STATE_INIT_CONNECTION; // Resend the request. -} - -int HttpNetworkTransaction::ReconsiderProxyAfterError(int error) { -#ifdef ANDROID - // Android crashes :( - return error; -#endif - DCHECK(!pac_request_); - - // A failure to resolve the hostname or any error related to establishing a - // TCP connection could be grounds for trying a new proxy configuration. - // - // Why do this when a hostname cannot be resolved? Some URLs only make sense - // to proxy servers. The hostname in those URLs might fail to resolve if we - // are still using a non-proxy config. We need to check if a proxy config - // now exists that corresponds to a proxy server that could load the URL. - // - switch (error) { - case ERR_NAME_NOT_RESOLVED: - case ERR_INTERNET_DISCONNECTED: - case ERR_ADDRESS_UNREACHABLE: - case ERR_CONNECTION_CLOSED: - case ERR_CONNECTION_RESET: - case ERR_CONNECTION_REFUSED: - case ERR_CONNECTION_ABORTED: - case ERR_TIMED_OUT: - case ERR_TUNNEL_CONNECTION_FAILED: - case ERR_SOCKS_CONNECTION_FAILED: - break; - case ERR_SOCKS_CONNECTION_HOST_UNREACHABLE: - // Remap the SOCKS-specific "host unreachable" error to a more - // generic error code (this way consumers like the link doctor - // know to substitute their error page). - // - // Note that if the host resolving was done by the SOCSK5 proxy, we can't - // differentiate between a proxy-side "host not found" versus a proxy-side - // "address unreachable" error, and will report both of these failures as - // ERR_ADDRESS_UNREACHABLE. - return ERR_ADDRESS_UNREACHABLE; - default: - return error; - } - - if (request_->load_flags & LOAD_BYPASS_PROXY) { - return error; - } - - int rv = session_->proxy_service()->ReconsiderProxyAfterError( - request_->url, &proxy_info_, &io_callback_, &pac_request_, net_log_); - if (rv == OK || rv == ERR_IO_PENDING) { - // If the error was during connection setup, there is no socket to - // disconnect. - if (connection_->socket()) - connection_->socket()->Disconnect(); - connection_->Reset(); - next_state_ = STATE_RESOLVE_PROXY_COMPLETE; - } else { - // If ReconsiderProxyAfterError() failed synchronously, it means - // there was nothing left to fall-back to, so fail the transaction - // with the last connection error we got. - // TODO(eroman): This is a confusing contract, make it more obvious. - rv = error; - } - - return rv; -======= next_state_ = STATE_CREATE_STREAM; // Resend the request. ->>>>>>> Chromium at release 7.0.540.0 } bool HttpNetworkTransaction::ShouldApplyProxyAuth() const { |