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

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

Issue 384873002: This CL changes the lifespan of SSLConnectJobMessengers so that they are created only when needed, (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@useloop
Patch Set: Fixed copiler issue in SSLClientSocketNSS Created 6 years, 4 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/socket/ssl_client_socket_pool.cc ('k') | no next file » | 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 #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 8
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/lazy_instance.h" 10 #include "base/lazy_instance.h"
(...skipping 374 matching lines...) Expand 10 before | Expand all | Expand 10 after
385 fallback_addresses_.reset(); 385 fallback_addresses_.reset();
386 } 386 }
387 NotifyDelegateOfCompletion(result); // Deletes |this| 387 NotifyDelegateOfCompletion(result); // Deletes |this|
388 } 388 }
389 389
390 int TransportConnectJob::ConnectInternal() { 390 int TransportConnectJob::ConnectInternal() {
391 return helper_.DoConnectInternal(this); 391 return helper_.DoConnectInternal(this);
392 } 392 }
393 393
394 scoped_ptr<ConnectJob> 394 scoped_ptr<ConnectJob>
395 TransportClientSocketPool::TransportConnectJobFactory::NewConnectJob( 395 TransportClientSocketPool::TransportConnectJobFactory::NewConnectJob(
396 const std::string& group_name, 396 const std::string& group_name,
397 const PoolBase::Request& request, 397 const PoolBase::Request& request,
398 ConnectJob::Delegate* delegate) const { 398 ConnectJob::Delegate* delegate) const {
399 return scoped_ptr<ConnectJob>( 399 return scoped_ptr<ConnectJob>(
400 new TransportConnectJob(group_name, 400 new TransportConnectJob(group_name,
401 request.priority(), 401 request.priority(),
402 request.params(), 402 request.params(),
403 ConnectionTimeout(), 403 ConnectionTimeout(),
404 client_socket_factory_, 404 client_socket_factory_,
405 host_resolver_, 405 host_resolver_,
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
535 HigherLayeredPool* higher_pool) { 535 HigherLayeredPool* higher_pool) {
536 base_.AddHigherLayeredPool(higher_pool); 536 base_.AddHigherLayeredPool(higher_pool);
537 } 537 }
538 538
539 void TransportClientSocketPool::RemoveHigherLayeredPool( 539 void TransportClientSocketPool::RemoveHigherLayeredPool(
540 HigherLayeredPool* higher_pool) { 540 HigherLayeredPool* higher_pool) {
541 base_.RemoveHigherLayeredPool(higher_pool); 541 base_.RemoveHigherLayeredPool(higher_pool);
542 } 542 }
543 543
544 } // namespace net 544 } // namespace net
OLDNEW
« no previous file with comments | « net/socket/ssl_client_socket_pool.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698