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

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

Issue 8591037: Implement Drain() on HttpPipelinedStream. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Skip on unusable Created 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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_pipelined_stream.h" 5 #include "net/http/http_pipelined_stream.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/stringprintf.h" 8 #include "base/stringprintf.h"
9 #include "net/base/net_errors.h" 9 #include "net/base/net_errors.h"
10 #include "net/http/http_pipelined_connection_impl.h" 10 #include "net/http/http_pipelined_connection_impl.h"
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
111 } 111 }
112 112
113 bool HttpPipelinedStream::IsSpdyHttpStream() const { 113 bool HttpPipelinedStream::IsSpdyHttpStream() const {
114 return false; 114 return false;
115 } 115 }
116 116
117 void HttpPipelinedStream::LogNumRttVsBytesMetrics() const { 117 void HttpPipelinedStream::LogNumRttVsBytesMetrics() const {
118 // TODO(simonjam): I don't want to copy & paste this from http_basic_stream. 118 // TODO(simonjam): I don't want to copy & paste this from http_basic_stream.
119 } 119 }
120 120
121 void HttpPipelinedStream::Drain(HttpNetworkSession*) { 121 void HttpPipelinedStream::Drain(HttpNetworkSession* session) {
122 // On errors, we already evict everything from the pipeline and close it. 122 pipeline_->Drain(this, session);
123 // TODO(simonjam): Consider trying to drain the pipeline in the same way that
124 // HttpBasicStream does.
125 delete this;
126 } 123 }
127 124
128 const SSLConfig& HttpPipelinedStream::used_ssl_config() const { 125 const SSLConfig& HttpPipelinedStream::used_ssl_config() const {
129 return pipeline_->used_ssl_config(); 126 return pipeline_->used_ssl_config();
130 } 127 }
131 128
132 const ProxyInfo& HttpPipelinedStream::used_proxy_info() const { 129 const ProxyInfo& HttpPipelinedStream::used_proxy_info() const {
133 return pipeline_->used_proxy_info(); 130 return pipeline_->used_proxy_info();
134 } 131 }
135 132
136 const NetLog::Source& HttpPipelinedStream::source() const { 133 const NetLog::Source& HttpPipelinedStream::source() const {
137 return pipeline_->source(); 134 return pipeline_->source();
138 } 135 }
139 136
140 bool HttpPipelinedStream::was_npn_negotiated() const { 137 bool HttpPipelinedStream::was_npn_negotiated() const {
141 return pipeline_->was_npn_negotiated(); 138 return pipeline_->was_npn_negotiated();
142 } 139 }
143 140
144 } // namespace net 141 } // namespace net
OLDNEW
« no previous file with comments | « net/http/http_pipelined_network_transaction_unittest.cc ('k') | net/http/http_response_body_drainer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698