Virtual destructors should have virtual keyword.

chrome/browser/debugger/inspectable_tab_proxy.h

 // Copyright (c) 2009 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 CHROME_BROWSER_DEBUGGER_INSPECTABLE_TAB_PROXY_H_
 #define CHROME_BROWSER_DEBUGGER_INSPECTABLE_TAB_PROXY_H_
 #pragma once
 
 #include <string>
 
@@ skipping 47 matching lines @@
 
 
 // An internal implementation of DevToolsClientHost that delegates
 // messages sent for DevToolsClient to a DebuggerShell instance.
 class DevToolsClientHostImpl : public DevToolsClientHost {
  public:
   DevToolsClientHostImpl(
     int32 id,
     DebuggerRemoteService* service,
     InspectableTabProxy::IdToClientHostMap* map);
-  ~DevToolsClientHostImpl();
+  virtual ~DevToolsClientHostImpl();
 
   DebuggerRemoteService* debugger_remote_service() {
     return service_;
   }
 
   void Close();
 
   // DevToolsClientHost interface
   virtual void InspectedTabClosing();
   virtual void SendMessageToClient(const IPC::Message& msg);
   virtual void TabReplaced(TabContentsWrapper* new_tab);
 
  private:
   // Message handling routines
   void OnDebuggerOutput(const std::string& msg);
   virtual void FrameNavigating(const std::string& url);
   void TabClosed();
 
   int32 id_;
   DebuggerRemoteService* service_;
   InspectableTabProxy::IdToClientHostMap* map_;
 };
 
 #endif  // CHROME_BROWSER_DEBUGGER_INSPECTABLE_TAB_PROXY_H_