Rework the logging for sockets/connectjobs....

net/socket/client_socket_handle.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/client_socket_handle.h"
 
 #include "base/compiler_specific.h"
 #include "base/histogram.h"
 #include "base/logging.h"
 #include "net/base/net_errors.h"
@@ skipping 15 matching lines @@
 void ClientSocketHandle::Reset() {
   ResetInternal(true);
 }
 
 void ClientSocketHandle::ResetInternal(bool cancel) {
   if (group_name_.empty())  // Was Init called?
     return;
   if (socket_.get()) {
     // Because of http://crbug.com/37810 we may not have a pool, but have
     // just a raw socket.
+    socket_->NetLog().EndEvent(NetLog::TYPE_SOCKET_IN_USE, NULL);
     if (pool_)
       // If we've still got a socket, release it back to the ClientSocketPool so
       // it can be deleted or reused.
       pool_->ReleaseSocket(group_name_, release_socket());
   } else if (cancel) {
     // If we did not get initialized yet, so we've got a socket request pending.
     // Cancel it.
     pool_->CancelRequest(group_name_, this);
   }
   group_name_.clear();
@@ skipping 39 matching lines @@
     case ClientSocketHandle::UNUSED_IDLE:
       histograms->AddUnusedIdleTime(idle_time());
       break;
     case ClientSocketHandle::REUSED_IDLE:
       histograms->AddReusedIdleTime(idle_time());
       break;
     default:
       NOTREACHED();
       break;
   }
+
+  // Broadcast that the socket has been acquired.
+  // TODO(eroman): This logging is not complete, in particular set_socket() and
+  // release() socket. It ends up working though, since those methods are being
+  // used to layer sockets (and the destination sources are the same).
+  DCHECK(socket_.get());
+  socket_->NetLog().BeginEvent(
+      NetLog::TYPE_SOCKET_IN_USE,
+      new NetLogSourceParameter("source_dependency", requesting_source_));
 }
 
 }  // namespace net