RefCounted types should not have public destructors, content/ remaining bits

content/plugin/plugin_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_PLUGIN_PLUGIN_CHANNEL_H_
 #define CONTENT_PLUGIN_PLUGIN_CHANNEL_H_
 #pragma once
 
 #include <vector>
 #include "base/memory/ref_counted.h"
@@ skipping 13 matching lines @@
  public:
   // Get a new PluginChannel object for the current process to talk to the
   // given renderer process. The renderer ID is an opaque unique ID generated
   // by the browser.
   static PluginChannel* GetPluginChannel(
       int renderer_id, base::MessageLoopProxy* ipc_message_loop);
 
   // Send a message to all renderers that the process is going to shutdown.
   static void NotifyRenderersOfPendingShutdown();
 
-  virtual ~PluginChannel();
-
+  // IPC::Channel::Listener:
   virtual bool Send(IPC::Message* msg) OVERRIDE;
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
+  virtual void OnChannelError() OVERRIDE;
 
   int renderer_id() { return renderer_id_; }
 
   virtual int GenerateRouteID() OVERRIDE;
 
   // Returns the event that's set when a call to the renderer causes a modal
   // dialog to come up.
   virtual base::WaitableEvent* GetModalDialogEvent(
       gfx::NativeViewId containing_window) OVERRIDE;
 
   bool in_send() { return in_send_ != 0; }
 
   bool incognito() { return incognito_; }
   void set_incognito(bool value) { incognito_ = value; }
 
 #if defined(OS_POSIX)
   int TakeRendererFileDescriptor() {
     return channel_->TakeClientFileDescriptor();
   }
 #endif
 
  protected:
-  // IPC::Channel::Listener implementation:
-  virtual void OnChannelError() OVERRIDE;
+  virtual ~PluginChannel();
 
+  // NPChannelBase::
   virtual void CleanUp() OVERRIDE;
-
-  // Overrides NPChannelBase::Init.
   virtual bool Init(base::MessageLoopProxy* ipc_message_loop,
                     bool create_pipe_now,
                     base::WaitableEvent* shutdown_event) OVERRIDE;
 
  private:
   class MessageFilter;
 
   // Called on the plugin thread
   PluginChannel();
 
@@ skipping 15 matching lines @@
 
   int in_send_;  // Tracks if we're in a Send call.
   bool log_messages_;  // True if we should log sent and received messages.
   bool incognito_; // True if the renderer is in incognito mode.
   scoped_refptr<MessageFilter> filter_;  // Handles the modal dialog events.
 
   DISALLOW_COPY_AND_ASSIGN(PluginChannel);
 };
 
 #endif  // CONTENT_PLUGIN_PLUGIN_CHANNEL_H_