Refcounted types should not have public destructors, remoting/ edition

remoting/protocol/message_reader.h

Index: remoting/protocol/message_reader.h
diff --git a/remoting/protocol/message_reader.h b/remoting/protocol/message_reader.h
index a7c1c005b0995d745d151818a90a628bef51b843..a2cfca3761ae72969f010aa84edde3ad55e85915 100644
--- a/remoting/protocol/message_reader.h
+++ b/remoting/protocol/message_reader.h
@@ -41,13 +41,15 @@ class MessageReader : public base::RefCountedThreadSafe<MessageReader> {
       MessageReceivedCallback;
 
   MessageReader();
-  virtual ~MessageReader();
 
   // Initialize the MessageReader with a socket. If a message is received
   // |callback| is called.
   void Init(net::Socket* socket, const MessageReceivedCallback& callback);
 
  private:
+  friend class base::RefCountedThreadSafe<MessageReader>;
+  virtual ~MessageReader();
+
   void DoRead();
   void OnRead(int result);
   void HandleReadResult(int result);