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

content/plugin/plugin_thread.cc

 // 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.
 
 #include "content/plugin/plugin_thread.h"
 
 #include "build/build_config.h"
 
 #if defined(TOOLKIT_GTK)
 #include <gtk/gtk.h>
@@ skipping 26 matching lines @@
 
 #if defined(USE_X11)
 #include "ui/base/x/x11_util.h"
 #endif
 
 namespace {
 
 class EnsureTerminateMessageFilter : public IPC::ChannelProxy::MessageFilter {
  public:
   EnsureTerminateMessageFilter() {}
-  ~EnsureTerminateMessageFilter() {}
 
- private:
+ protected:
+  virtual ~EnsureTerminateMessageFilter() {}
+
+  // IPC::ChannelProxy::MessageFilter:
   virtual void OnChannelError() {
     // How long we wait before forcibly shutting down the process.
     const base::TimeDelta kPluginProcessTerminateTimeout =
         base::TimeDelta::FromSeconds(3);
     // Ensure that we don't wait indefinitely for the plugin to shutdown.
     // as the browser does not terminate plugin processes on shutdown.
     // We achieve this by posting an exit process task on the IO thread.
     MessageLoop::current()->PostDelayedTask(
         FROM_HERE,
         base::Bind(&EnsureTerminateMessageFilter::Terminate, this),
         kPluginProcessTerminateTimeout);
   }
 
+ private:
   void Terminate() {
     base::KillProcess(base::GetCurrentProcessHandle(), 0, false);
   }
 };
 
 }  // namespace
 
 static base::LazyInstance<base::ThreadLocalPointer<PluginThread> > lazy_tls =
     LAZY_INSTANCE_INITIALIZER;
 
@@ skipping 100 matching lines @@
 #endif
     channel->set_incognito(incognito);
   }
 
   Send(new PluginProcessHostMsg_ChannelCreated(channel_handle));
 }
 
 void PluginThread::OnNotifyRenderersOfPendingShutdown() {
   PluginChannel::NotifyRenderersOfPendingShutdown();
 }