RefCounted types should not have public destructors, delegate cleanup

content/browser/ppapi_plugin_process_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_BROWSER_PPAPI_PLUGIN_PROCESS_HOST_H_
 #define CONTENT_BROWSER_PPAPI_PLUGIN_PROCESS_HOST_H_
 #pragma once
 
 #include <string>
 #include <queue>
@@ skipping 34 matching lines @@
     // error. On error, the parameters should be:
     //   base::kNullProcessHandle
     //   IPC::ChannelHandle(),
     //   0
     virtual void OnPpapiChannelOpened(
         const IPC::ChannelHandle& channel_handle,
         int plugin_child_id) = 0;
 
     // Returns true if the current connection is off-the-record.
     virtual bool OffTheRecord() = 0;
+
+   protected:
+    virtual ~Client() {}
   };
 
   class PluginClient : public Client {
    public:
     // Returns the resource context for the renderer requesting the channel.
     virtual content::ResourceContext* GetResourceContext() = 0;
+
+   protected:
+    virtual ~PluginClient() {}
   };
 
   class BrokerClient : public Client {
+   protected:
+    virtual ~BrokerClient() {}
   };
 
   virtual ~PpapiPluginProcessHost();
 
   static PpapiPluginProcessHost* CreatePluginHost(
       const content::PepperPluginInfo& info,
       const FilePath& profile_data_directory,
       net::HostResolver* host_resolver);
   static PpapiPluginProcessHost* CreateBrokerHost(
       const content::PepperPluginInfo& info);
@@ skipping 82 matching lines @@
     : public content::BrowserChildProcessHostTypeIterator<
           PpapiPluginProcessHost> {
  public:
   PpapiBrokerProcessHostIterator()
       : content::BrowserChildProcessHostTypeIterator<
           PpapiPluginProcessHost>(content::PROCESS_TYPE_PPAPI_BROKER) {}
 };
 
 #endif  // CONTENT_BROWSER_PPAPI_PLUGIN_PROCESS_HOST_H_