RefCounted types should not have public destructors, content/browser part 2

content/renderer/p2p/socket_dispatcher.cc

 // Copyright (c) 2012 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 "content/renderer/p2p/socket_dispatcher.h"
 
 #include "base/bind.h"
 #include "base/memory/ref_counted.h"
 #include "base/message_loop_proxy.h"
 #include "content/common/p2p_messages.h"
@@ skipping 16 matching lines @@
     DCHECK(message_loop_->BelongsToCurrentThread());
     message_sender_ = NULL;
   }
 
   void Send(IPC::Message* msg) {
     message_loop_->PostTask(FROM_HERE,
                             base::Bind(&AsyncMessageSender::DoSend, this, msg));
   }
 
  private:
+  friend class base::RefCountedThreadSafe<AsyncMessageSender>;
+  ~AsyncMessageSender() {}
+
   void DoSend(IPC::Message* msg) {
     DCHECK(message_loop_->BelongsToCurrentThread());
     if (message_sender_)
       message_sender_->Send(msg);
   }
 
   scoped_refptr<base::MessageLoopProxy> message_loop_;
   IPC::Message::Sender* message_sender_;
 
   DISALLOW_COPY_AND_ASSIGN(AsyncMessageSender);
@@ skipping 129 matching lines @@
     // hasn't processed the close message by the time it sends the
     // message to the renderer.
     VLOG(1) << "Received P2P message for socket that doesn't exist.";
     return NULL;
   }
 
   return client;
 }
 
 }  // namespace content