Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(278)

Side by Side Diff: net/http/http_stream_factory_impl_job.cc

Issue 2348453002: Remove NPN from SSLConfig and SSLClientSocket*. (Closed)
Patch Set: Re: #7. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « net/http/http_server_properties.cc ('k') | net/log/net_log_util.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/http/http_stream_factory_impl_job.h" 5 #include "net/http/http_stream_factory_impl_job.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 925 matching lines...) Expand 10 before | Expand all | Expand 10 after
936 // the hostname is resolved. 936 // the hostname is resolved.
937 OnHostResolutionCallback resolution_callback = 937 OnHostResolutionCallback resolution_callback =
938 CanUseExistingSpdySession() 938 CanUseExistingSpdySession()
939 ? base::Bind(&Job::OnHostResolution, session_->spdy_session_pool(), 939 ? base::Bind(&Job::OnHostResolution, session_->spdy_session_pool(),
940 spdy_session_key, origin_url_) 940 spdy_session_key, origin_url_)
941 : OnHostResolutionCallback(); 941 : OnHostResolutionCallback();
942 if (delegate_->for_websockets()) { 942 if (delegate_->for_websockets()) {
943 // TODO(ricea): Re-enable NPN when WebSockets over SPDY is supported. 943 // TODO(ricea): Re-enable NPN when WebSockets over SPDY is supported.
944 SSLConfig websocket_server_ssl_config = server_ssl_config_; 944 SSLConfig websocket_server_ssl_config = server_ssl_config_;
945 websocket_server_ssl_config.alpn_protos.clear(); 945 websocket_server_ssl_config.alpn_protos.clear();
946 websocket_server_ssl_config.npn_protos.clear();
947 return InitSocketHandleForWebSocketRequest( 946 return InitSocketHandleForWebSocketRequest(
948 GetSocketGroup(), destination_, request_info_.extra_headers, 947 GetSocketGroup(), destination_, request_info_.extra_headers,
949 request_info_.load_flags, priority_, session_, proxy_info_, expect_spdy, 948 request_info_.load_flags, priority_, session_, proxy_info_, expect_spdy,
950 websocket_server_ssl_config, proxy_ssl_config_, 949 websocket_server_ssl_config, proxy_ssl_config_,
951 request_info_.privacy_mode, net_log_, connection_.get(), 950 request_info_.privacy_mode, net_log_, connection_.get(),
952 resolution_callback, io_callback_); 951 resolution_callback, io_callback_);
953 } 952 }
954 953
955 return InitSocketHandleForHttpRequest( 954 return InitSocketHandleForHttpRequest(
956 GetSocketGroup(), destination_, request_info_.extra_headers, 955 GetSocketGroup(), destination_, request_info_.extra_headers,
(...skipping 565 matching lines...) Expand 10 before | Expand all | Expand 10 after
1522 1521
1523 ConnectionAttempts socket_attempts = connection_->connection_attempts(); 1522 ConnectionAttempts socket_attempts = connection_->connection_attempts();
1524 if (connection_->socket()) { 1523 if (connection_->socket()) {
1525 connection_->socket()->GetConnectionAttempts(&socket_attempts); 1524 connection_->socket()->GetConnectionAttempts(&socket_attempts);
1526 } 1525 }
1527 1526
1528 delegate_->AddConnectionAttemptsToRequest(this, socket_attempts); 1527 delegate_->AddConnectionAttemptsToRequest(this, socket_attempts);
1529 } 1528 }
1530 1529
1531 } // namespace net 1530 } // namespace net
OLDNEW
« no previous file with comments | « net/http/http_server_properties.cc ('k') | net/log/net_log_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698