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

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

Issue 1326503003: Added a net::BidirectionalStream to expose a bidirectional streaming interface (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix bug and added a test Created 5 years 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
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_network_transaction.h" 5 #include "net/http/http_network_transaction.h"
6 6
7 #include <set> 7 #include <set>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 498 matching lines...) Expand 10 before | Expand all | Expand 10 after
509 response_.was_npn_negotiated = stream_request_->was_npn_negotiated(); 509 response_.was_npn_negotiated = stream_request_->was_npn_negotiated();
510 response_.npn_negotiated_protocol = SSLClientSocket::NextProtoToString( 510 response_.npn_negotiated_protocol = SSLClientSocket::NextProtoToString(
511 stream_request_->protocol_negotiated()); 511 stream_request_->protocol_negotiated());
512 response_.was_fetched_via_spdy = stream_request_->using_spdy(); 512 response_.was_fetched_via_spdy = stream_request_->using_spdy();
513 response_.was_fetched_via_proxy = !proxy_info_.is_direct(); 513 response_.was_fetched_via_proxy = !proxy_info_.is_direct();
514 if (response_.was_fetched_via_proxy && !proxy_info_.is_empty()) 514 if (response_.was_fetched_via_proxy && !proxy_info_.is_empty())
515 response_.proxy_server = proxy_info_.proxy_server().host_port_pair(); 515 response_.proxy_server = proxy_info_.proxy_server().host_port_pair();
516 OnIOComplete(OK); 516 OnIOComplete(OK);
517 } 517 }
518 518
519 void HttpNetworkTransaction::OnBidirectionalStreamJobReady(
520 const SSLConfig& used_ssl_config,
521 const ProxyInfo& used_proxy_info,
522 BidirectionalStreamJob* stream) {
523 NOTREACHED();
524 }
525
519 void HttpNetworkTransaction::OnWebSocketHandshakeStreamReady( 526 void HttpNetworkTransaction::OnWebSocketHandshakeStreamReady(
520 const SSLConfig& used_ssl_config, 527 const SSLConfig& used_ssl_config,
521 const ProxyInfo& used_proxy_info, 528 const ProxyInfo& used_proxy_info,
522 WebSocketHandshakeStreamBase* stream) { 529 WebSocketHandshakeStreamBase* stream) {
523 OnStreamReady(used_ssl_config, used_proxy_info, stream); 530 OnStreamReady(used_ssl_config, used_proxy_info, stream);
524 } 531 }
525 532
526 void HttpNetworkTransaction::OnStreamFailed(int result, 533 void HttpNetworkTransaction::OnStreamFailed(int result,
527 const SSLConfig& used_ssl_config, 534 const SSLConfig& used_ssl_config,
528 SSLFailureState ssl_failure_state) { 535 SSLFailureState ssl_failure_state) {
(...skipping 1152 matching lines...) Expand 10 before | Expand all | Expand 10 after
1681 DCHECK(stream_request_); 1688 DCHECK(stream_request_);
1682 1689
1683 // Since the transaction can restart with auth credentials, it may create a 1690 // Since the transaction can restart with auth credentials, it may create a
1684 // stream more than once. Accumulate all of the connection attempts across 1691 // stream more than once. Accumulate all of the connection attempts across
1685 // those streams by appending them to the vector: 1692 // those streams by appending them to the vector:
1686 for (const auto& attempt : stream_request_->connection_attempts()) 1693 for (const auto& attempt : stream_request_->connection_attempts())
1687 connection_attempts_.push_back(attempt); 1694 connection_attempts_.push_back(attempt);
1688 } 1695 }
1689 1696
1690 } // namespace net 1697 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698