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

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

Issue 2600943002: Cleanup the preconnect to proxy code and Job controller code (Closed)
Patch Set: rch comments Created 3 years, 11 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_stream_factory_impl.cc ('k') | net/http/http_stream_factory_impl_job.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 #ifndef NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_ 5 #ifndef NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_
6 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_ 6 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 // An HttpStreamRequestImpl exists for each stream which is in progress of being 44 // An HttpStreamRequestImpl exists for each stream which is in progress of being
45 // created for the StreamFactory. 45 // created for the StreamFactory.
46 class HttpStreamFactoryImpl::Job { 46 class HttpStreamFactoryImpl::Job {
47 public: 47 public:
48 // Delegate to report Job's status to Request and HttpStreamFactory. 48 // Delegate to report Job's status to Request and HttpStreamFactory.
49 class NET_EXPORT_PRIVATE Delegate { 49 class NET_EXPORT_PRIVATE Delegate {
50 public: 50 public:
51 virtual ~Delegate() {} 51 virtual ~Delegate() {}
52 52
53 // Invoked when |job| has an HttpStream ready. 53 // Invoked when |job| has an HttpStream ready.
54 virtual void OnStreamReady(Job* job, 54 virtual void OnStreamReady(Job* job, const SSLConfig& used_ssl_config) = 0;
55 const SSLConfig& used_ssl_config,
56 const ProxyInfo& used_proxy_info) = 0;
57 55
58 // Invoked when |job| has a BidirectionalStream ready. 56 // Invoked when |job| has a BidirectionalStream ready.
59 virtual void OnBidirectionalStreamImplReady( 57 virtual void OnBidirectionalStreamImplReady(
60 Job* job, 58 Job* job,
61 const SSLConfig& used_ssl_config, 59 const SSLConfig& used_ssl_config,
62 const ProxyInfo& used_proxy_info) = 0; 60 const ProxyInfo& used_proxy_info) = 0;
63 61
64 // Invoked when |job| has a WebSocketHandshakeStream ready. 62 // Invoked when |job| has a WebSocketHandshakeStream ready.
65 virtual void OnWebSocketHandshakeStreamReady( 63 virtual void OnWebSocketHandshakeStreamReady(
66 Job* job, 64 Job* job,
(...skipping 458 matching lines...) Expand 10 before | Expand all | Expand 10 after
525 const SSLConfig& server_ssl_config, 523 const SSLConfig& server_ssl_config,
526 const SSLConfig& proxy_ssl_config, 524 const SSLConfig& proxy_ssl_config,
527 HostPortPair destination, 525 HostPortPair destination,
528 GURL origin_url, 526 GURL origin_url,
529 NetLog* net_log) = 0; 527 NetLog* net_log) = 0;
530 }; 528 };
531 529
532 } // namespace net 530 } // namespace net
533 531
534 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_ 532 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_
OLDNEW
« no previous file with comments | « net/http/http_stream_factory_impl.cc ('k') | net/http/http_stream_factory_impl_job.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698