Refcounted types should not have public destructors, remoting/ edition

remoting/host/screen_recorder.h

 // 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.
 
 #ifndef REMOTING_HOST_SCREEN_RECORDER_H_
 #define REMOTING_HOST_SCREEN_RECORDER_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 66 matching lines @@
  public:
 
   // Construct a ScreenRecorder. Message loops and threads are provided.
   // This object does not own capturer but owns encoder.
   ScreenRecorder(MessageLoop* capture_loop,
                  MessageLoop* encode_loop,
                  base::MessageLoopProxy* network_loop,
                  Capturer* capturer,
                  Encoder* encoder);
 
-  virtual ~ScreenRecorder();
-
   // Start recording.
   void Start();
 
   // Stop the recording session. |done_task| is executed when recording is fully
   // stopped. This object cannot be used again after |task| is executed.
   void Stop(const base::Closure& done_task);
 
   // Add a connection to this recording session.
   void AddConnection(protocol::ConnectionToClient* connection);
 
   // Remove a connection from receiving screen updates.
   void RemoveConnection(protocol::ConnectionToClient* connection);
 
   // Remove all connections.
   void RemoveAllConnections();
 
   // Update the sequence number for tracing performance.
   void UpdateSequenceNumber(int64 sequence_number);
 
  private:
+  friend class base::RefCountedThreadSafe<ScreenRecorder>;
+  virtual ~ScreenRecorder();
+
   // Getters for capturer and encoder.
   Capturer* capturer();
   Encoder* encoder();
 
   bool is_recording();
 
   // Capturer thread ----------------------------------------------------------
 
   void DoStart();
   void DoSetMaxRate(double max_rate);
@@ skipping 77 matching lines @@
 
   // An object to schedule capturing.
   CaptureScheduler scheduler_;
 
   DISALLOW_COPY_AND_ASSIGN(ScreenRecorder);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_SCREEN_RECORDER_H_