Deciding best connection to schedule requests on based on cwnd and idle time

net/socket/client_socket_pool_base.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // A ClientSocketPoolBase is used to restrict the number of sockets open at
 // a time.  It also maintains a list of idle persistent sockets for reuse.
 // Subclasses of ClientSocketPool should compose ClientSocketPoolBase to handle
 // the core logic of (1) restricting the number of active (connected or
 // connecting) sockets per "group" (generally speaking, the hostname), (2)
 // maintaining a per-group list of idle, persistent sockets for reuse, and (3)
@@ skipping 32 matching lines @@
 #include "net/base/net_log.h"
 #include "net/base/network_change_notifier.h"
 #include "net/base/request_priority.h"
 #include "net/socket/client_socket_pool.h"
 #include "net/socket/stream_socket.h"
 
 namespace net {
 
 class ClientSocketHandle;
 
+// Sets the client socket reuse policy (using warmest socket vs. last accessed
+// socket).
+// NOTE: 'policy' should be a valid ClientSocketReusePolicy enum value.
+double SetSocketReusePolicy(int policy);

[willchan no longer on Chromium, 2011/06/14 12:01:22]

This looks really weird. What does the return value do?

You also need to use NET_API since you're calling this from outside the net/
module. Otherwise the multi-dll build will break.

[Gagan, 2011/06/14 18:25:02]

Done.

+
 // ConnectJob provides an abstract interface for "connecting" a socket.
 // The connection may involve host resolution, tcp connection, ssl connection,
 // etc.
 class NET_TEST ConnectJob {
  public:
   class NET_TEST Delegate {
    public:
     Delegate() {}
     virtual ~Delegate() {}
 
@@ skipping 97 matching lines @@
       public NetworkChangeNotifier::IPAddressObserver {
  public:
   typedef uint32 Flags;
 
   // Used to specify specific behavior for the ClientSocketPool.
   enum Flag {
     NORMAL = 0,  // Normal behavior.
     NO_IDLE_SOCKETS = 0x1,  // Do not return an idle socket. Create a new one.
   };
 
+  enum ClientSocketReusePolicy {

[willchan no longer on Chromium, 2011/06/14 12:01:22]

It's not clear what the difference is between WARMEST and WARM socket. You need
comments.

[Gagan, 2011/06/14 18:25:02]

Done.

+    USE_WARMEST_SOCKET = 0,
+    USE_WARM_SOCKET = 1,
+    USE_LAST_ACCESSED_SOCKET = 2,
+  };
+
   class NET_TEST Request {
    public:
     Request(ClientSocketHandle* handle,
             CompletionCallback* callback,
             RequestPriority priority,
             bool ignore_limits,
             Flags flags,
             const BoundNetLog& net_log);
 
     virtual ~Request();
@@ skipping 566 matching lines @@
   // Histograms for the pool
   ClientSocketPoolHistograms* const histograms_;
   internal::ClientSocketPoolBaseHelper helper_;
 
   DISALLOW_COPY_AND_ASSIGN(ClientSocketPoolBase);
 };
 
 }  // namespace net
 
 #endif  // NET_SOCKET_CLIENT_SOCKET_POOL_BASE_H_