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

content/common/gpu/client/gpu_channel_host.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 CONTENT_COMMON_GPU_CLIENT_GPU_CHANNEL_HOST_H_
 #define CONTENT_COMMON_GPU_CLIENT_GPU_CHANNEL_HOST_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 67 matching lines @@
     kConnected,
     // An error caused the host to become disconnected. Recreate channel to
     // reestablish connection.
     kLost
   };
 
   // Called on the render thread
   GpuChannelHost(GpuChannelHostFactory* factory,
                  int gpu_process_id,
                  int client_id);
-  virtual ~GpuChannelHost();
 
   // Connect to GPU process channel.
   void Connect(const IPC::ChannelHandle& channel_handle,
                base::ProcessHandle client_process_for_gpu);
 
   State state() const { return state_; }
 
   // Change state to kLost.
   void SetStateLost();
 
@@ skipping 50 matching lines @@
   // cause all command buffers on this side to soon afterward start
   // registering lost contexts. It also has the side effect of setting
   // the state on this side to lost.
   void ForciblyCloseChannel();
 
   GpuChannelHostFactory* factory() const { return factory_; }
   int gpu_process_id() const { return gpu_process_id_; }
   int client_id() const { return client_id_; }
 
  private:
+  friend class base::RefCountedThreadSafe<GpuChannelHost>;
+  virtual ~GpuChannelHost();
+
   // A filter used internally to route incoming messages from the IO thread
   // to the correct message loop.
   class MessageFilter : public IPC::ChannelProxy::MessageFilter {
    public:
     explicit MessageFilter(GpuChannelHost* parent);
-    virtual ~MessageFilter();
 
     void AddRoute(int route_id,
                   base::WeakPtr<IPC::Channel::Listener> listener,
                   scoped_refptr<base::MessageLoopProxy> loop);
     void RemoveRoute(int route_id);
 
     // IPC::ChannelProxy::MessageFilter implementation:
     virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE;
     virtual void OnChannelError() OVERRIDE;
 
    private:
+    virtual ~MessageFilter();
+
     GpuChannelHost* parent_;
 
     typedef base::hash_map<int, GpuListenerInfo> ListenerMap;
     ListenerMap listeners_;
   };
 
   GpuChannelHostFactory* factory_;
   int gpu_process_id_;
   int client_id_;
 
@@ skipping 12 matching lines @@
   // for calls from contexts that may live on the compositor or main thread.
   mutable base::Lock context_lock_;
 
   // A filter for sending messages from thread other than the main thread.
   scoped_refptr<IPC::SyncMessageFilter> sync_filter_;
 
   DISALLOW_COPY_AND_ASSIGN(GpuChannelHost);
 };
 
 #endif  // CONTENT_COMMON_GPU_CLIENT_GPU_CHANNEL_HOST_H_