Virtual destructors should have virtual keyword.

content/browser/file_system/file_system_dispatcher_host.h

 // Copyright (c) 2011 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_FILE_SYSTEM_FILE_SYSTEM_DISPATCHER_HOST_H_
 #define CONTENT_BROWSER_FILE_SYSTEM_FILE_SYSTEM_DISPATCHER_HOST_H_
 
 #include <set>
 
 #include "base/basictypes.h"
@@ skipping 24 matching lines @@
 }  // namespace net
 
 class FileSystemDispatcherHost : public BrowserMessageFilter {
  public:
   // Used by the renderer.
   explicit FileSystemDispatcherHost(
       const content::ResourceContext* resource_context);
   // 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);
   virtual bool OnMessageReceived(const IPC::Message& message,
                                  bool* message_was_ok);
 
   void UnregisterOperation(int request_id);
 
  private:
   void OnOpen(int request_id,
@@ skipping 39 matching lines @@
 
   // This holds the ResourceContext until Init() can be called from the
   // IO thread, which will extract the net::URLRequestContext from it.
   const content::ResourceContext* resource_context_;
   net::URLRequestContext* request_context_;
 
   DISALLOW_COPY_AND_ASSIGN(FileSystemDispatcherHost);
 };
 
 #endif  // CONTENT_BROWSER_FILE_SYSTEM_FILE_SYSTEM_DISPATCHER_HOST_H_