Virtual destructors should have virtual keyword.

content/common/resource_dispatcher.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.
 
 // See http://dev.chromium.org/developers/design-documents/multi-process-resource-loading
 
 #ifndef CONTENT_COMMON_RESOURCE_DISPATCHER_H_
 #define CONTENT_COMMON_RESOURCE_DISPATCHER_H_
 #pragma once
 
@@ skipping 26 matching lines @@
         ResourceType::Type resource_type,
         const net::URLRequestStatus& status) = 0;
 
     virtual webkit_glue::ResourceLoaderBridge::Peer* OnReceivedResponse(
         webkit_glue::ResourceLoaderBridge::Peer* current_peer,
         const std::string& mime_type,
         const GURL& url) = 0;
   };
 
   explicit ResourceDispatcher(IPC::Message::Sender* sender);
-  ~ResourceDispatcher();
+  virtual ~ResourceDispatcher();
 
   // IPC::Channel::Listener implementation.
   virtual bool OnMessageReceived(const IPC::Message& message);
 
   // Creates a ResourceLoaderBridge for this type of dispatcher, this is so
   // this can be tested regardless of the ResourceLoaderBridge::Create
   // implementation.
   webkit_glue::ResourceLoaderBridge* CreateBridge(
       const webkit_glue::ResourceLoaderBridge::RequestInfo& request_info);
 
@@ skipping 107 matching lines @@
   PendingRequestList pending_requests_;
 
   ScopedRunnableMethodFactory<ResourceDispatcher> method_factory_;
 
   scoped_ptr<Observer> observer_;
 
   DISALLOW_COPY_AND_ASSIGN(ResourceDispatcher);
 };
 
 #endif  // CONTENT_COMMON_RESOURCE_DISPATCHER_H_