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

Side by Side Diff: net/http/http_stream_factory.h

Issue 2893713004: Return Request as unique_ptr from JobController::Start(). (Closed)
Patch Set: Fix conflict with https://crrev.com/2887773006. Created 3 years, 7 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_network_transaction_unittest.cc ('k') | net/http/http_stream_factory_impl.h » ('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 #ifndef NET_HTTP_HTTP_STREAM_FACTORY_H_ 5 #ifndef NET_HTTP_HTTP_STREAM_FACTORY_H_
6 #define NET_HTTP_HTTP_STREAM_FACTORY_H_ 6 #define NET_HTTP_HTTP_STREAM_FACTORY_H_
7 7
8 #include <list> 8 #include <list>
9 #include <memory> 9 #include <memory>
10 #include <string> 10 #include <string>
(...skipping 185 matching lines...) Expand 10 before | Expand all | Expand 10 after
196 virtual ~HttpStreamFactory(); 196 virtual ~HttpStreamFactory();
197 197
198 void ProcessAlternativeServices(HttpNetworkSession* session, 198 void ProcessAlternativeServices(HttpNetworkSession* session,
199 const HttpResponseHeaders* headers, 199 const HttpResponseHeaders* headers,
200 const url::SchemeHostPort& http_server); 200 const url::SchemeHostPort& http_server);
201 201
202 // Virtual interface methods. 202 // Virtual interface methods.
203 203
204 // Request a stream. 204 // Request a stream.
205 // Will call delegate->OnStreamReady on successful completion. 205 // Will call delegate->OnStreamReady on successful completion.
206 virtual HttpStreamRequest* RequestStream( 206 virtual std::unique_ptr<HttpStreamRequest> RequestStream(
207 const HttpRequestInfo& info, 207 const HttpRequestInfo& info,
208 RequestPriority priority, 208 RequestPriority priority,
209 const SSLConfig& server_ssl_config, 209 const SSLConfig& server_ssl_config,
210 const SSLConfig& proxy_ssl_config, 210 const SSLConfig& proxy_ssl_config,
211 HttpStreamRequest::Delegate* delegate, 211 HttpStreamRequest::Delegate* delegate,
212 bool enable_ip_based_pooling, 212 bool enable_ip_based_pooling,
213 bool enable_alternative_services, 213 bool enable_alternative_services,
214 const NetLogWithSource& net_log) = 0; 214 const NetLogWithSource& net_log) = 0;
215 215
216 // Request a WebSocket handshake stream. 216 // Request a WebSocket handshake stream.
217 // Will call delegate->OnWebSocketHandshakeStreamReady on successful 217 // Will call delegate->OnWebSocketHandshakeStreamReady on successful
218 // completion. 218 // completion.
219 virtual HttpStreamRequest* RequestWebSocketHandshakeStream( 219 virtual std::unique_ptr<HttpStreamRequest> RequestWebSocketHandshakeStream(
220 const HttpRequestInfo& info, 220 const HttpRequestInfo& info,
221 RequestPriority priority, 221 RequestPriority priority,
222 const SSLConfig& server_ssl_config, 222 const SSLConfig& server_ssl_config,
223 const SSLConfig& proxy_ssl_config, 223 const SSLConfig& proxy_ssl_config,
224 HttpStreamRequest::Delegate* delegate, 224 HttpStreamRequest::Delegate* delegate,
225 WebSocketHandshakeStreamBase::CreateHelper* create_helper, 225 WebSocketHandshakeStreamBase::CreateHelper* create_helper,
226 bool enable_ip_based_pooling, 226 bool enable_ip_based_pooling,
227 bool enable_alternative_services, 227 bool enable_alternative_services,
228 const NetLogWithSource& net_log) = 0; 228 const NetLogWithSource& net_log) = 0;
229 229
230 // Request a BidirectionalStreamImpl. 230 // Request a BidirectionalStreamImpl.
231 // Will call delegate->OnBidirectionalStreamImplReady on successful 231 // Will call delegate->OnBidirectionalStreamImplReady on successful
232 // completion. 232 // completion.
233 virtual HttpStreamRequest* RequestBidirectionalStreamImpl( 233 virtual std::unique_ptr<HttpStreamRequest> RequestBidirectionalStreamImpl(
234 const HttpRequestInfo& info, 234 const HttpRequestInfo& info,
235 RequestPriority priority, 235 RequestPriority priority,
236 const SSLConfig& server_ssl_config, 236 const SSLConfig& server_ssl_config,
237 const SSLConfig& proxy_ssl_config, 237 const SSLConfig& proxy_ssl_config,
238 HttpStreamRequest::Delegate* delegate, 238 HttpStreamRequest::Delegate* delegate,
239 bool enable_ip_based_pooling, 239 bool enable_ip_based_pooling,
240 bool enable_alternative_services, 240 bool enable_alternative_services,
241 const NetLogWithSource& net_log) = 0; 241 const NetLogWithSource& net_log) = 0;
242 242
243 // Requests that enough connections for |num_streams| be opened. 243 // Requests that enough connections for |num_streams| be opened.
(...skipping 13 matching lines...) Expand all
257 257
258 private: 258 private:
259 url::SchemeHostPort RewriteHost(const url::SchemeHostPort& server); 259 url::SchemeHostPort RewriteHost(const url::SchemeHostPort& server);
260 260
261 DISALLOW_COPY_AND_ASSIGN(HttpStreamFactory); 261 DISALLOW_COPY_AND_ASSIGN(HttpStreamFactory);
262 }; 262 };
263 263
264 } // namespace net 264 } // namespace net
265 265
266 #endif // NET_HTTP_HTTP_STREAM_FACTORY_H_ 266 #endif // NET_HTTP_HTTP_STREAM_FACTORY_H_
OLDNEW
« no previous file with comments | « net/http/http_network_transaction_unittest.cc ('k') | net/http/http_stream_factory_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698