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

content/common/gpu/gpu_channel.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_GPU_CHANNEL_H_
 #define CONTENT_COMMON_GPU_GPU_CHANNEL_H_
 #pragma once
 
 #include <deque>
 #include <string>
@@ skipping 27 matching lines @@
 class GpuChannel : public IPC::Channel::Listener,
                    public IPC::Message::Sender,
                    public base::RefCountedThreadSafe<GpuChannel> {
  public:
   // Takes ownership of the renderer process handle.
   GpuChannel(GpuChannelManager* gpu_channel_manager,
              GpuWatchdog* watchdog,
              gfx::GLShareGroup* share_group,
              int client_id,
              bool software);
-  virtual ~GpuChannel();
 
   bool Init(base::MessageLoopProxy* io_message_loop,
             base::WaitableEvent* shutdown_event);
 
   // Get the GpuChannelManager that owns this channel.
   GpuChannelManager* gpu_channel_manager() const {
     return gpu_channel_manager_;
   }
 
   // Returns the name of the associated IPC channel.
@@ skipping 40 matching lines @@
   int GenerateRouteID();
 
   // Called to add/remove a listener for a particular message routing ID.
   void AddRoute(int32 route_id, IPC::Channel::Listener* listener);
   void RemoveRoute(int32 route_id);
 
   // Indicates whether newly created contexts should prefer the
   // discrete GPU even if they would otherwise use the integrated GPU.
   bool ShouldPreferDiscreteGpu() const;
 
+ protected:
+  virtual ~GpuChannel();
+
  private:
+  friend class base::RefCountedThreadSafe<GpuChannel>;
+
   void OnDestroy();
 
   bool OnControlMessageReceived(const IPC::Message& msg);
 
   void HandleMessage();
   void PollWork(int route_id);
   void ScheduleDelayedWork(GpuCommandBufferStub *stub, int64 delay);
 
   // Message handlers.
   void OnCreateOffscreenCommandBuffer(
@@ skipping 44 matching lines @@
   bool handle_messages_scheduled_;
   bool processed_get_state_fast_;
   int32 num_contexts_preferring_discrete_gpu_;
 
   base::WeakPtrFactory<GpuChannel> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(GpuChannel);
 };
 
 #endif  // CONTENT_COMMON_GPU_GPU_CHANNEL_H_