Cleanup: Address some of the todos in net_log.h...

net/socket/tcp_client_socket_pool.cc

 // Copyright (c) 2010 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.
 
 #include "net/socket/tcp_client_socket_pool.h"
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/string_util.h"
@@ skipping 182 matching lines @@
     const std::string& group_name,
     const void* params,
     RequestPriority priority,
     ClientSocketHandle* handle,
     CompletionCallback* callback,
     const BoundNetLog& net_log) {
   const TCPSocketParams* casted_params =
       static_cast<const TCPSocketParams*>(params);
 
   if (net_log.HasListener()) {
-    net_log.AddString(StringPrintf("Requested TCP socket to: %s [port %d]",
-                      casted_params->destination().hostname().c_str(),
-                      casted_params->destination().port()));
+    // TODO(eroman): Split out the host and port parameters.
+    net_log.AddEventWithString(
+        NetLog::TYPE_TCP_CLIENT_SOCKET_POOL_REQUESTED_SOCKET,
+        "host_and_port",
+        StringPrintf("%s [port %d]",
+                     casted_params->destination().hostname().c_str(),
+                     casted_params->destination().port()));
   }
 
   return base_.RequestSocket(group_name, *casted_params, priority, handle,
                              callback, net_log);
 }
 
 void TCPClientSocketPool::CancelRequest(
     const std::string& group_name,
     const ClientSocketHandle* handle) {
   base_.CancelRequest(group_name, handle);
@@ skipping 13 matching lines @@
     const std::string& group_name) const {
   return base_.IdleSocketCountInGroup(group_name);
 }
 
 LoadState TCPClientSocketPool::GetLoadState(
     const std::string& group_name, const ClientSocketHandle* handle) const {
   return base_.GetLoadState(group_name, handle);
 }
 
 }  // namespace net