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

Side by Side Diff: net/socket/transport_client_socket_pool.cc

Issue 2537373002: Use different source types for each ConnectJob subclass. (Closed)
Patch Set: mmenke comment Created 4 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/socket/transport_client_socket_pool.h" 5 #include "net/socket/transport_client_socket_pool.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 SocketPerformanceWatcherFactory* socket_performance_watcher_factory, 94 SocketPerformanceWatcherFactory* socket_performance_watcher_factory,
95 HostResolver* host_resolver, 95 HostResolver* host_resolver,
96 Delegate* delegate, 96 Delegate* delegate,
97 NetLog* net_log) 97 NetLog* net_log)
98 : ConnectJob( 98 : ConnectJob(
99 group_name, 99 group_name,
100 timeout_duration, 100 timeout_duration,
101 priority, 101 priority,
102 respect_limits, 102 respect_limits,
103 delegate, 103 delegate,
104 NetLogWithSource::Make(net_log, NetLogSourceType::CONNECT_JOB)), 104 NetLogWithSource::Make(net_log,
105 NetLogSourceType::TRANSPORT_CONNECT_JOB)),
105 params_(params), 106 params_(params),
106 resolver_(host_resolver), 107 resolver_(host_resolver),
107 client_socket_factory_(client_socket_factory), 108 client_socket_factory_(client_socket_factory),
108 next_state_(STATE_NONE), 109 next_state_(STATE_NONE),
109 socket_performance_watcher_factory_(socket_performance_watcher_factory), 110 socket_performance_watcher_factory_(socket_performance_watcher_factory),
110 resolve_result_(OK) {} 111 resolve_result_(OK) {}
111 112
112 TransportConnectJob::~TransportConnectJob() { 113 TransportConnectJob::~TransportConnectJob() {
113 // We don't worry about cancelling the host resolution and TCP connect, since 114 // We don't worry about cancelling the host resolution and TCP connect, since
114 // ~HostResolver::Request and ~StreamSocket will take care of it. 115 // ~HostResolver::Request and ~StreamSocket will take care of it.
(...skipping 479 matching lines...) Expand 10 before | Expand all | Expand 10 after
594 HigherLayeredPool* higher_pool) { 595 HigherLayeredPool* higher_pool) {
595 base_.AddHigherLayeredPool(higher_pool); 596 base_.AddHigherLayeredPool(higher_pool);
596 } 597 }
597 598
598 void TransportClientSocketPool::RemoveHigherLayeredPool( 599 void TransportClientSocketPool::RemoveHigherLayeredPool(
599 HigherLayeredPool* higher_pool) { 600 HigherLayeredPool* higher_pool) {
600 base_.RemoveHigherLayeredPool(higher_pool); 601 base_.RemoveHigherLayeredPool(higher_pool);
601 } 602 }
602 603
603 } // namespace net 604 } // namespace net
OLDNEW
« no previous file with comments | « net/socket/ssl_client_socket_pool.cc ('k') | net/socket/websocket_transport_client_socket_pool.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698