Refcounted types should not have public destructors, remoting/ edition

remoting/protocol/buffered_socket_writer.h

Index: remoting/protocol/buffered_socket_writer.h
diff --git a/remoting/protocol/buffered_socket_writer.h b/remoting/protocol/buffered_socket_writer.h
index 7bc1b09e3aa628a85a7545737fc5500288399dc4..53eb08f06e9a0ca6a7db58956e2a6bb8ecb6c541 100644
--- a/remoting/protocol/buffered_socket_writer.h
+++ b/remoting/protocol/buffered_socket_writer.h
@@ -39,7 +39,6 @@ class BufferedSocketWriterBase
   typedef base::Callback<void(int)> WriteFailedCallback;
 
   explicit BufferedSocketWriterBase(base::MessageLoopProxy* message_loop);
-  virtual ~BufferedSocketWriterBase();
 
   // Initializes the writer. Must be called on the thread that will be used
   // to access the socket in the future. |callback| will be called after each
@@ -64,9 +63,13 @@ class BufferedSocketWriterBase
   void Close();
 
  protected:
+  friend class base::RefCountedThreadSafe<BufferedSocketWriterBase>;
+
   class PendingPacket;
   typedef std::list<PendingPacket*> DataQueue;
 
+  virtual ~BufferedSocketWriterBase();
+
   DataQueue queue_;
   int buffer_size_;
 
@@ -105,7 +108,6 @@ class BufferedSocketWriterBase
 class BufferedSocketWriter : public BufferedSocketWriterBase {
  public:
   explicit BufferedSocketWriter(base::MessageLoopProxy* message_loop);
-  virtual ~BufferedSocketWriter();
 
  protected:
   virtual void GetNextPacket_Locked(net::IOBuffer** buffer, int* size) OVERRIDE;
@@ -113,18 +115,22 @@ class BufferedSocketWriter : public BufferedSocketWriterBase {
   virtual void OnError_Locked(int result) OVERRIDE;
 
  private:
+  virtual ~BufferedSocketWriter();
+
   scoped_refptr<net::DrainableIOBuffer> current_buf_;
 };
 
 class BufferedDatagramWriter : public BufferedSocketWriterBase {
  public:
   explicit BufferedDatagramWriter(base::MessageLoopProxy* message_loop);
-  virtual ~BufferedDatagramWriter();
 
  protected:
   virtual void GetNextPacket_Locked(net::IOBuffer** buffer, int* size) OVERRIDE;
   virtual void AdvanceBufferPosition_Locked(int written) OVERRIDE;
   virtual void OnError_Locked(int result) OVERRIDE;
+
+ private:
+  virtual ~BufferedDatagramWriter();
 };
 
 }  // namespace protocol