Virtual destructors should have virtual keyword.

Make sure user-declared virtual destructors always have the virtual keyword.
The Clang style-check plugin will check for this soon.

No functionality change: virtual is only added
to destructors that are already implicitly virtual.

Also fix a couple of in-line destructor definitions.

BUG=83408
TEST=none

Review URL: http://codereview.chromium.org/7064033

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@86587 0039d316-1c4b-4281-b951-d872f2087c98

47 files changed, 47 insertions, 47 deletions

diff --git a/content/browser/appcache/appcache_dispatcher_host.h b/content/browser/appcache/appcache_dispatcher_host.h
index 776ef04..4799159 100644
--- a/content/browser/appcache/appcache_dispatcher_host.h
+++ b/content/browser/appcache/appcache_dispatcher_host.h
@@ -29,7 +29,7 @@ class AppCacheDispatcherHost : public BrowserMessageFilter {
  public:
   AppCacheDispatcherHost(const content::ResourceContext* resource_context,
                          int process_id);
-  ~AppCacheDispatcherHost();
+  virtual ~AppCacheDispatcherHost();
 
   // BrowserIOMessageFilter implementation
   virtual void OnChannelConnected(int32 peer_pid);
diff --git a/content/browser/cert_store.h b/content/browser/cert_store.h
index 7115ec7..af5e46c 100644
--- a/content/browser/cert_store.h
+++ b/content/browser/cert_store.h
@@ -50,7 +50,7 @@ class CertStore : public NotificationObserver {
   friend struct DefaultSingletonTraits<CertStore>;
 
   CertStore();
-  ~CertStore();
+  virtual ~CertStore();
 
   // Remove the specified cert from id_to_cert_ and cert_to_id_.
   // NOTE: the caller (RemoveCertsForRenderProcesHost) must hold cert_lock_.
diff --git a/content/browser/file_system/file_system_dispatcher_host.h b/content/browser/file_system/file_system_dispatcher_host.h
index 48867d5..ee7dac7 100644
--- a/content/browser/file_system/file_system_dispatcher_host.h
+++ b/content/browser/file_system/file_system_dispatcher_host.h
@@ -42,7 +42,7 @@ class FileSystemDispatcherHost : public BrowserMessageFilter {
   // Used by the worker, since it has the context handy already.
   FileSystemDispatcherHost(net::URLRequestContext* request_context,
                            fileapi::FileSystemContext* file_system_context);
-  ~FileSystemDispatcherHost();
+  virtual ~FileSystemDispatcherHost();
 
   // BrowserMessageFilter implementation.
   virtual void OnChannelConnected(int32 peer_pid);
diff --git a/content/browser/geolocation/core_location_provider_mac.h b/content/browser/geolocation/core_location_provider_mac.h
index d26428d..2c528ca 100644
--- a/content/browser/geolocation/core_location_provider_mac.h
+++ b/content/browser/geolocation/core_location_provider_mac.h
@@ -18,7 +18,7 @@ class CoreLocationDataProviderMac;
 class CoreLocationProviderMac : public LocationProviderBase {
  public:
   explicit CoreLocationProviderMac();
-  ~CoreLocationProviderMac();
+  virtual ~CoreLocationProviderMac();
 
   // LocationProvider
   virtual bool StartProvider(bool high_accuracy);
diff --git a/content/browser/geolocation/geolocation_provider.h b/content/browser/geolocation/geolocation_provider.h
index aaf398e..c955a3c 100644
--- a/content/browser/geolocation/geolocation_provider.h
+++ b/content/browser/geolocation/geolocation_provider.h
@@ -57,7 +57,7 @@ class GeolocationProvider : public base::Thread, public GeolocationObserver {
 
  private:
   friend struct DefaultSingletonTraits<GeolocationProvider>;
-  ~GeolocationProvider();
+  virtual ~GeolocationProvider();
 
   bool OnClientThread() const;
   bool OnGeolocationThread() const;
diff --git a/content/browser/geolocation/location_arbitrator.h b/content/browser/geolocation/location_arbitrator.h
index 264ca1a..84522eb 100644
--- a/content/browser/geolocation/location_arbitrator.h
+++ b/content/browser/geolocation/location_arbitrator.h
@@ -40,7 +40,7 @@ class GeolocationArbitrator : public LocationProviderBase::ListenerInterface {
   // Defines a function that returns the current time.
   typedef base::Time (*GetTimeNow)();
 
-  ~GeolocationArbitrator();
+  virtual ~GeolocationArbitrator();
 
   static GeolocationArbitrator* Create(GeolocationObserver* observer);
 
diff --git a/content/browser/geolocation/mock_location_provider.h b/content/browser/geolocation/mock_location_provider.h
index 9dfb51d..4bfee4a 100644
--- a/content/browser/geolocation/mock_location_provider.h
+++ b/content/browser/geolocation/mock_location_provider.h
@@ -20,7 +20,7 @@ class MockLocationProvider : public LocationProviderBase {
   // Will update |*self_ref| to point to |this| on construction, and to NULL
   // on destruction.
   explicit MockLocationProvider(MockLocationProvider** self_ref);
-  ~MockLocationProvider();
+  virtual ~MockLocationProvider();
 
   // Updates listeners with the new position.
   void HandlePositionChanged(const Geoposition& position);
diff --git a/content/browser/gpu/gpu_process_host_ui_shim.h b/content/browser/gpu/gpu_process_host_ui_shim.h
index d0782d7..fc077fa 100644
--- a/content/browser/gpu/gpu_process_host_ui_shim.h
+++ b/content/browser/gpu/gpu_process_host_ui_shim.h
@@ -40,7 +40,7 @@ class Message;
 class RouteToGpuProcessHostUIShimTask : public Task {
  public:
   RouteToGpuProcessHostUIShimTask(int host_id, const IPC::Message& msg);
-  ~RouteToGpuProcessHostUIShimTask();
+  virtual ~RouteToGpuProcessHostUIShimTask();
 
  private:
   virtual void Run();
diff --git a/content/browser/host_zoom_map.h b/content/browser/host_zoom_map.h
index 3ae44a1..5a923e3 100644
--- a/content/browser/host_zoom_map.h
+++ b/content/browser/host_zoom_map.h
@@ -77,7 +77,7 @@ class HostZoomMap :
 
   typedef std::map<std::string, double> HostZoomLevels;
 
-  ~HostZoomMap();
+  virtual ~HostZoomMap();
 
   // Copy of the pref data, so that we can read it on the IO thread.
   HostZoomLevels host_zoom_levels_;
diff --git a/content/browser/in_process_webkit/dom_storage_message_filter.h b/content/browser/in_process_webkit/dom_storage_message_filter.h
index e994d98..3976d7c 100644
--- a/content/browser/in_process_webkit/dom_storage_message_filter.h
+++ b/content/browser/in_process_webkit/dom_storage_message_filter.h
@@ -42,7 +42,7 @@ class DOMStorageMessageFilter : public BrowserMessageFilter {
  private:
   friend class BrowserThread;
   friend class DeleteTask<DOMStorageMessageFilter>;
-  ~DOMStorageMessageFilter();
+  virtual ~DOMStorageMessageFilter();
 
   // Message Handlers.
   void OnStorageAreaId(int64 namespace_id, const string16& origin,
diff --git a/content/browser/in_process_webkit/indexed_db_dispatcher_host.h b/content/browser/in_process_webkit/indexed_db_dispatcher_host.h
index fbe8bfc..49a93fb 100644
--- a/content/browser/in_process_webkit/indexed_db_dispatcher_host.h
+++ b/content/browser/in_process_webkit/indexed_db_dispatcher_host.h
@@ -59,7 +59,7 @@ class IndexedDBDispatcherHost : public BrowserMessageFilter {
   int32 Add(WebKit::WebIDBTransaction* idb_transaction);
 
  private:
-  ~IndexedDBDispatcherHost();
+  virtual ~IndexedDBDispatcherHost();
 
   // Message processing. Most of the work is delegated to the dispatcher hosts
   // below.
diff --git a/content/browser/mime_registry_message_filter.h b/content/browser/mime_registry_message_filter.h
index 8342458..cdb6a86 100644
--- a/content/browser/mime_registry_message_filter.h
+++ b/content/browser/mime_registry_message_filter.h
@@ -18,7 +18,7 @@ class MimeRegistryMessageFilter : public BrowserMessageFilter {
                                  bool* message_was_ok);
 
  private:
-  ~MimeRegistryMessageFilter();
+  virtual ~MimeRegistryMessageFilter();
 
   void OnGetMimeTypeFromExtension(const FilePath::StringType& ext,
                                   std::string* mime_type);
diff --git a/content/browser/mock_resource_context.h b/content/browser/mock_resource_context.h
index 89a619a..26e81d8 100644
--- a/content/browser/mock_resource_context.h
+++ b/content/browser/mock_resource_context.h
@@ -24,7 +24,7 @@ class MockResourceContext : public ResourceContext {
   friend struct base::DefaultLazyInstanceTraits<MockResourceContext>;
 
   MockResourceContext();
-  ~MockResourceContext();
+  virtual ~MockResourceContext();
   virtual void EnsureInitialized() const OVERRIDE;
 
   const scoped_refptr<net::URLRequestContext> test_request_context_;
diff --git a/content/browser/plugin_service.h b/content/browser/plugin_service.h
index 2c8d115..cc039d6 100644
--- a/content/browser/plugin_service.h
+++ b/content/browser/plugin_service.h
@@ -124,7 +124,7 @@ class PluginService
   // Creates the PluginService object, but doesn't actually build the plugin
   // list yet.  It's generated lazily.
   PluginService();
-  ~PluginService();
+  virtual ~PluginService();
 
   // base::WaitableEventWatcher::Delegate implementation.
   virtual void OnWaitableEventSignaled(base::WaitableEvent* waitable_event);
diff --git a/content/browser/renderer_host/blob_message_filter.h b/content/browser/renderer_host/blob_message_filter.h
index bb8a191..ace01db 100644
--- a/content/browser/renderer_host/blob_message_filter.h
+++ b/content/browser/renderer_host/blob_message_filter.h
@@ -23,7 +23,7 @@ class BlobMessageFilter : public BrowserMessageFilter {
  public:
   BlobMessageFilter(int process_id,
                     ChromeBlobStorageContext* blob_storage_context);
-  ~BlobMessageFilter();
+  virtual ~BlobMessageFilter();
 
   // BrowserMessageFilter implementation.
   virtual void OnChannelClosing();
diff --git a/content/browser/renderer_host/browser_render_process_host.h b/content/browser/renderer_host/browser_render_process_host.h
index 9a5874f..f8b63de 100644
--- a/content/browser/renderer_host/browser_render_process_host.h
+++ b/content/browser/renderer_host/browser_render_process_host.h
@@ -44,7 +44,7 @@ class BrowserRenderProcessHost : public RenderProcessHost,
                                  public ChildProcessLauncher::Client {
  public:
   explicit BrowserRenderProcessHost(Profile* profile);
-  ~BrowserRenderProcessHost();
+  virtual ~BrowserRenderProcessHost();
 
   // RenderProcessHost implementation (public portion).
   virtual bool Init(bool is_accessibility_enabled);
diff --git a/content/browser/renderer_host/clipboard_message_filter.h b/content/browser/renderer_host/clipboard_message_filter.h
index 3128852..48c0c65 100644
--- a/content/browser/renderer_host/clipboard_message_filter.h
+++ b/content/browser/renderer_host/clipboard_message_filter.h
@@ -23,7 +23,7 @@ class ClipboardMessageFilter : public BrowserMessageFilter {
   virtual bool OnMessageReceived(const IPC::Message& message,
                                  bool* message_was_ok);
  private:
-  ~ClipboardMessageFilter();
+  virtual ~ClipboardMessageFilter();
 
   void OnWriteObjectsAsync(const ui::Clipboard::ObjectMap& objects);
   void OnWriteObjectsSync(const ui::Clipboard::ObjectMap& objects,
diff --git a/content/browser/renderer_host/file_utilities_message_filter.h b/content/browser/renderer_host/file_utilities_message_filter.h
index add9e95..1003a6fe 100644
--- a/content/browser/renderer_host/file_utilities_message_filter.h
+++ b/content/browser/renderer_host/file_utilities_message_filter.h
@@ -28,7 +28,7 @@ class FileUtilitiesMessageFilter : public BrowserMessageFilter {
   virtual bool OnMessageReceived(const IPC::Message& message,
                                  bool* message_was_ok);
  private:
-  ~FileUtilitiesMessageFilter();
+  virtual ~FileUtilitiesMessageFilter();
 
   typedef void (*FileInfoWriteFunc)(IPC::Message* reply_msg,
                                     const base::PlatformFileInfo& file_info);
diff --git a/content/browser/renderer_host/quota_dispatcher_host.h b/content/browser/renderer_host/quota_dispatcher_host.h
index 5a9fd4a..d1fa405 100644
--- a/content/browser/renderer_host/quota_dispatcher_host.h
+++ b/content/browser/renderer_host/quota_dispatcher_host.h
@@ -23,7 +23,7 @@ class QuotaManager;
 class QuotaDispatcherHost : public BrowserMessageFilter {
  public:
   QuotaDispatcherHost(quota::QuotaManager* quota_manager);
-  ~QuotaDispatcherHost();
+  virtual ~QuotaDispatcherHost();
   virtual bool OnMessageReceived(const IPC::Message& message,
                                  bool* message_was_ok);
 
diff --git a/content/browser/renderer_host/resource_dispatcher_host.h b/content/browser/renderer_host/resource_dispatcher_host.h
index f8643fe..3e007ef 100644
--- a/content/browser/renderer_host/resource_dispatcher_host.h
+++ b/content/browser/renderer_host/resource_dispatcher_host.h
@@ -61,7 +61,7 @@ class ResourceDispatcherHost : public net::URLRequest::Delegate {
  public:
   explicit ResourceDispatcherHost(
       const ResourceQueue::DelegateSet& resource_queue_delegates);
-  ~ResourceDispatcherHost();
+  virtual ~ResourceDispatcherHost();
 
   void Initialize();
 
diff --git a/content/browser/renderer_host/sync_resource_handler.h b/content/browser/renderer_host/sync_resource_handler.h
index c07064c..24267da 100644
--- a/content/browser/renderer_host/sync_resource_handler.h
+++ b/content/browser/renderer_host/sync_resource_handler.h
@@ -47,7 +47,7 @@ class SyncResourceHandler : public ResourceHandler {
  private:
   enum { kReadBufSize = 3840 };
 
-  ~SyncResourceHandler();
+  virtual ~SyncResourceHandler();
 
   scoped_refptr<net::IOBuffer> read_buffer_;
 
diff --git a/content/browser/resolve_proxy_msg_helper.h b/content/browser/resolve_proxy_msg_helper.h
index 5032687..85b3b69 100644
--- a/content/browser/resolve_proxy_msg_helper.h
+++ b/content/browser/resolve_proxy_msg_helper.h
@@ -33,7 +33,7 @@ class ResolveProxyMsgHelper : public BrowserMessageFilter {
 
   // Destruction cancels the current outstanding request, and clears the
   // pending queue.
-  ~ResolveProxyMsgHelper();
+  virtual ~ResolveProxyMsgHelper();
 
   // BrowserMessageFilter implementation
   virtual bool OnMessageReceived(const IPC::Message& message,
diff --git a/content/browser/speech/speech_recognizer.h b/content/browser/speech/speech_recognizer.h
index f60a9de..f49b3ea 100644
--- a/content/browser/speech/speech_recognizer.h
+++ b/content/browser/speech/speech_recognizer.h
@@ -84,7 +84,7 @@ class SpeechRecognizer
                    const std::string& grammar,
                    const std::string& hardware_info,
                    const std::string& origin_url);
-  ~SpeechRecognizer();
+  virtual ~SpeechRecognizer();
 
   // Starts audio recording and does recognition after recording ends. The same
   // SpeechRecognizer instance can be used multiple times for speech recognition
diff --git a/content/browser/worker_host/worker_message_filter.h b/content/browser/worker_host/worker_message_filter.h
index 52915e1..d5556f8 100644
--- a/content/browser/worker_host/worker_message_filter.h
+++ b/content/browser/worker_host/worker_message_filter.h
@@ -41,7 +41,7 @@ class WorkerMessageFilter : public BrowserMessageFilter {
   }
 
  private:
-  ~WorkerMessageFilter();
+  virtual ~WorkerMessageFilter();
 
   // Message handlers.
   void OnCreateWorker(const ViewHostMsg_CreateWorker_Params& params,
diff --git a/content/browser/worker_host/worker_process_host.h b/content/browser/worker_host/worker_process_host.h
index 29637b0..068624b 100644
--- a/content/browser/worker_host/worker_process_host.h
+++ b/content/browser/worker_host/worker_process_host.h
@@ -117,7 +117,7 @@ class WorkerProcessHost : public BrowserChildProcessHost {
   WorkerProcessHost(
       const content::ResourceContext* resource_context,
       ResourceDispatcherHost* resource_dispatcher_host);
-  ~WorkerProcessHost();
+  virtual ~WorkerProcessHost();
 
   // Starts the process.  Returns true iff it succeeded.
   // |render_process_id| is the renderer process responsible for starting this
diff --git a/content/common/file_system/file_system_dispatcher.h b/content/common/file_system/file_system_dispatcher.h
index b480ce8..fe01733 100644
--- a/content/common/file_system/file_system_dispatcher.h
+++ b/content/common/file_system/file_system_dispatcher.h
@@ -30,7 +30,7 @@ class GURL;
 class FileSystemDispatcher : public IPC::Channel::Listener {
  public:
   FileSystemDispatcher();
-  ~FileSystemDispatcher();
+  virtual ~FileSystemDispatcher();
 
   // IPC::Channel::Listener implementation.
   virtual bool OnMessageReceived(const IPC::Message& msg);
diff --git a/content/common/font_config_ipc_linux.h b/content/common/font_config_ipc_linux.h
index 4873c14..ed23c28 100644
--- a/content/common/font_config_ipc_linux.h
+++ b/content/common/font_config_ipc_linux.h
@@ -15,7 +15,7 @@
 class FontConfigIPC : public FontConfigInterface {
  public:
   explicit FontConfigIPC(int fd);
-  ~FontConfigIPC();
+  virtual ~FontConfigIPC();
 
   // FontConfigInterface implementation.
   virtual bool Match(std::string* result_family,
diff --git a/content/common/gpu/gpu_channel_manager.h b/content/common/gpu/gpu_channel_manager.h
index f031f76e..a163597 100644
--- a/content/common/gpu/gpu_channel_manager.h
+++ b/content/common/gpu/gpu_channel_manager.h
@@ -47,7 +47,7 @@ class GpuChannelManager : public IPC::Channel::Listener,
                     GpuWatchdog* watchdog,
                     base::MessageLoopProxy* io_message_loop,
                     base::WaitableEvent* shutdown_event);
-  ~GpuChannelManager();
+  virtual ~GpuChannelManager();
 
   // Remove the channel for a particular renderer.
   void RemoveChannel(int renderer_id);
diff --git a/content/common/quota_dispatcher.h b/content/common/quota_dispatcher.h
index d283b28..ddc2b79 100644
--- a/content/common/quota_dispatcher.h
+++ b/content/common/quota_dispatcher.h
@@ -30,7 +30,7 @@ class WebStorageQuotaCallbacks;
 class QuotaDispatcher : public IPC::Channel::Listener {
  public:
   QuotaDispatcher();
-  ~QuotaDispatcher();
+  virtual ~QuotaDispatcher();
 
   // IPC::Channel::Listener implementation.
   virtual bool OnMessageReceived(const IPC::Message& msg);
diff --git a/content/common/resource_dispatcher.h b/content/common/resource_dispatcher.h
index 8317774..0c1a5cc 100644
--- a/content/common/resource_dispatcher.h
+++ b/content/common/resource_dispatcher.h
@@ -44,7 +44,7 @@ class ResourceDispatcher : public IPC::Channel::Listener {
   };
 
   explicit ResourceDispatcher(IPC::Message::Sender* sender);
-  ~ResourceDispatcher();
+  virtual ~ResourceDispatcher();
 
   // IPC::Channel::Listener implementation.
   virtual bool OnMessageReceived(const IPC::Message& message);
diff --git a/content/gpu/gpu_child_thread.h b/content/gpu/gpu_child_thread.h
index 94a9bdf..6fc8fbd 100644
--- a/content/gpu/gpu_child_thread.h
+++ b/content/gpu/gpu_child_thread.h
@@ -47,7 +47,7 @@ class GpuChildThread : public ChildThread {
   // For single-process mode.
   explicit GpuChildThread(const std::string& channel_id);
 
-  ~GpuChildThread();
+  virtual ~GpuChildThread();
 
   void Init(const base::Time& process_start_time);
   void StopWatchdog();
diff --git a/content/gpu/gpu_process.h b/content/gpu/gpu_process.h
index 8dfe0e3..bd05a72 100644
--- a/content/gpu/gpu_process.h
+++ b/content/gpu/gpu_process.h
@@ -11,7 +11,7 @@
 class GpuProcess : public ChildProcess {
  public:
   GpuProcess();
-  ~GpuProcess();
+  virtual ~GpuProcess();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(GpuProcess);
diff --git a/content/plugin/npobject_proxy.h b/content/plugin/npobject_proxy.h
index 68bca78..dccd531 100644
--- a/content/plugin/npobject_proxy.h
+++ b/content/plugin/npobject_proxy.h
@@ -31,7 +31,7 @@ class NPObjectProxy : public IPC::Channel::Listener,
                       public IPC::Message::Sender,
                       public NPObjectBase {
  public:
-  ~NPObjectProxy();
+  virtual ~NPObjectProxy();
 
   static NPObject* Create(PluginChannelBase* channel,
                           int route_id,
diff --git a/content/plugin/npobject_stub.h b/content/plugin/npobject_stub.h
index 46e1bc1..c100fe4 100644
--- a/content/plugin/npobject_stub.h
+++ b/content/plugin/npobject_stub.h
@@ -36,7 +36,7 @@ class NPObjectStub : public IPC::Channel::Listener,
                int route_id,
                gfx::NativeViewId containing_window,
                const GURL& page_url);
-  ~NPObjectStub();
+  virtual ~NPObjectStub();
 
   // IPC::Message::Sender implementation:
   virtual bool Send(IPC::Message* msg);
diff --git a/content/plugin/plugin_thread.h b/content/plugin/plugin_thread.h
index efd96b6..d325dd0 100644
--- a/content/plugin/plugin_thread.h
+++ b/content/plugin/plugin_thread.h
@@ -23,7 +23,7 @@
 class PluginThread : public ChildThread {
  public:
   PluginThread();
-  ~PluginThread();
+  virtual ~PluginThread();
 
   // Returns the one plugin thread.
   static PluginThread* current();
diff --git a/content/plugin/webplugin_delegate_stub.h b/content/plugin/webplugin_delegate_stub.h
index b602a04..629da0a 100644
--- a/content/plugin/webplugin_delegate_stub.h
+++ b/content/plugin/webplugin_delegate_stub.h
@@ -55,7 +55,7 @@ class WebPluginDelegateStub : public IPC::Channel::Listener,
  private:
   friend class base::RefCounted<WebPluginDelegateStub>;
 
-  ~WebPluginDelegateStub();
+  virtual ~WebPluginDelegateStub();
 
   // Message handlers for the WebPluginDelegate calls that are proxied from the
   // renderer over the IPC channel.
diff --git a/content/plugin/webplugin_proxy.h b/content/plugin/webplugin_proxy.h
index 47f3ab2..3cde1cb 100644
--- a/content/plugin/webplugin_proxy.h
+++ b/content/plugin/webplugin_proxy.h
@@ -48,7 +48,7 @@ class WebPluginProxy : public webkit::npapi::WebPlugin {
                  const GURL& page_url,
                  gfx::NativeViewId containing_window,
                  int host_render_view_routing_id);
-  ~WebPluginProxy();
+  virtual ~WebPluginProxy();
 
   void set_delegate(webkit::npapi::WebPluginDelegateImpl* d) { delegate_ = d; }
 
diff --git a/content/ppapi_plugin/ppapi_thread.h b/content/ppapi_plugin/ppapi_thread.h
index eedbd0a..a9d0b7c 100644
--- a/content/ppapi_plugin/ppapi_thread.h
+++ b/content/ppapi_plugin/ppapi_thread.h
@@ -27,7 +27,7 @@ class PpapiThread : public ChildThread,
                     public pp::proxy::PluginDispatcher::PluginDelegate {
  public:
   explicit PpapiThread(bool is_broker);
-  ~PpapiThread();
+  virtual ~PpapiThread();
 
  private:
   // ChildThread overrides.
diff --git a/content/renderer/audio_message_filter.h b/content/renderer/audio_message_filter.h
index 316ac1b..2403e8f 100644
--- a/content/renderer/audio_message_filter.h
+++ b/content/renderer/audio_message_filter.h
@@ -53,7 +53,7 @@ class AudioMessageFilter : public IPC::ChannelProxy::MessageFilter {
   };
 
   explicit AudioMessageFilter(int32 route_id);
-  ~AudioMessageFilter();
+  virtual ~AudioMessageFilter();
 
   // Add a delegate to the map and return id of the entry.
   int32 AddDelegate(Delegate* delegate);
diff --git a/content/renderer/gpu_channel_host.h b/content/renderer/gpu_channel_host.h
index ba7ec84..7e67da8 100644
--- a/content/renderer/gpu_channel_host.h
+++ b/content/renderer/gpu_channel_host.h
@@ -43,7 +43,7 @@ class GpuChannelHost : public IPC::Channel::Listener,
 
   // Called on the render thread
   GpuChannelHost();
-  ~GpuChannelHost();
+  virtual ~GpuChannelHost();
 
   // Connect to GPU process channel.
   void Connect(const IPC::ChannelHandle& channel_handle,
diff --git a/content/renderer/indexed_db_dispatcher.h b/content/renderer/indexed_db_dispatcher.h
index b019510..8bd608e 100644
--- a/content/renderer/indexed_db_dispatcher.h
+++ b/content/renderer/indexed_db_dispatcher.h
@@ -28,7 +28,7 @@ class WebIDBTransaction;
 class IndexedDBDispatcher : public IPC::Channel::Listener {
  public:
   IndexedDBDispatcher();
-  ~IndexedDBDispatcher();
+  virtual ~IndexedDBDispatcher();
 
   // IPC::Channel::Listener implementation.
   virtual bool OnMessageReceived(const IPC::Message& msg);
diff --git a/content/renderer/plugin_channel_host.h b/content/renderer/plugin_channel_host.h
index ec6bfed..524df47 100644
--- a/content/renderer/plugin_channel_host.h
+++ b/content/renderer/plugin_channel_host.h
@@ -46,7 +46,7 @@ class PluginChannelHost : public PluginChannelBase {
  private:
   // Called on the render thread
   PluginChannelHost();
-  ~PluginChannelHost();
+  virtual ~PluginChannelHost();
 
   static PluginChannelBase* ClassFactory() { return new PluginChannelHost(); }
 
diff --git a/content/renderer/render_process_impl.h b/content/renderer/render_process_impl.h
index b530e6a..c5cbabe 100644
--- a/content/renderer/render_process_impl.h
+++ b/content/renderer/render_process_impl.h
@@ -20,7 +20,7 @@ class PlatformCanvas;
 class RenderProcessImpl : public RenderProcess {
  public:
   RenderProcessImpl();
-  ~RenderProcessImpl();
+  virtual ~RenderProcessImpl();
 
   // RenderProcess implementation.
   virtual skia::PlatformCanvas* GetDrawingCanvas(TransportDIB** memory,
diff --git a/content/renderer/renderer_webidbobjectstore_impl.h b/content/renderer/renderer_webidbobjectstore_impl.h
index 802ba32..6467960 100644
--- a/content/renderer/renderer_webidbobjectstore_impl.h
+++ b/content/renderer/renderer_webidbobjectstore_impl.h
@@ -22,7 +22,7 @@ class WebString;
 class RendererWebIDBObjectStoreImpl : public WebKit::WebIDBObjectStore {
  public:
   explicit RendererWebIDBObjectStoreImpl(int32 idb_object_store_id);
-  ~RendererWebIDBObjectStoreImpl();
+  virtual ~RendererWebIDBObjectStoreImpl();
 
   // WebKit::WebIDBObjectStore
   virtual WebKit::WebString name() const;
diff --git a/content/renderer/webplugin_delegate_proxy.h b/content/renderer/webplugin_delegate_proxy.h
index 304e72b..7912436 100644
--- a/content/renderer/webplugin_delegate_proxy.h
+++ b/content/renderer/webplugin_delegate_proxy.h
@@ -123,7 +123,7 @@ class WebPluginDelegateProxy
 
  protected:
   template<class WebPluginDelegateProxy> friend class DeleteTask;
-  ~WebPluginDelegateProxy();
+  virtual ~WebPluginDelegateProxy();
 
  private:
   // Message handlers for messages that proxy WebPlugin methods, which
diff --git a/content/renderer/webworker_proxy.h b/content/renderer/webworker_proxy.h
index 9151193..80014ac 100644
--- a/content/renderer/webworker_proxy.h
+++ b/content/renderer/webworker_proxy.h
@@ -29,7 +29,7 @@ class WebWorkerProxy : public WebKit::WebWorker, private WebWorkerBase {
                  ChildThread* child_thread,
                  int render_view_route_id,
                  int parent_appcache_host_id);
-  ~WebWorkerProxy();
+  virtual ~WebWorkerProxy();
 
   // WebWorker implementation.
   virtual void startWorkerContext(const WebKit::WebURL& script_url,
diff --git a/content/worker/worker_thread.h b/content/worker/worker_thread.h
index 25995a6..4e0e487 100644
--- a/content/worker/worker_thread.h
+++ b/content/worker/worker_thread.h
@@ -21,7 +21,7 @@ struct WorkerProcessMsg_CreateWorker_Params;
 class WorkerThread : public ChildThread {
  public:
   WorkerThread();
-  ~WorkerThread();
+  virtual ~WorkerThread();
 
   // Returns the one worker thread.
   static WorkerThread* current();