Virtual destructors should have virtual keyword.

chrome/browser/tabs/tab_finder.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 CHROME_BROWSER_TABS_TAB_FINDER_H_
 #define CHROME_BROWSER_TABS_TAB_FINDER_H_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 37 matching lines @@
 
  private:
   friend struct DefaultSingletonTraits<TabFinder>;
 
   class TabContentsObserverImpl;
 
   typedef std::map<TabContents*, GURL> TabContentsToURLMap;
   typedef std::set<TabContentsObserverImpl*> TabContentsObservers;
 
   TabFinder();
-  ~TabFinder();
+  virtual ~TabFinder();
 
   void Init();
 
   // Forwarded from TabContentsObserverImpl.
   void DidNavigateAnyFramePostCommit(
       TabContents* source,
       const NavigationController::LoadCommittedDetails& details,
       const ViewHostMsg_FrameNavigate_Params& params);
 
   // Returns true if the tab's current url is |url|, or the start of the
@@ skipping 33 matching lines @@
   CancelableRequestConsumerTSimple<TabContents*> callback_consumer_;
 
   NotificationRegistrar registrar_;
 
   TabContentsObservers tab_contents_observers_;
 
   DISALLOW_COPY_AND_ASSIGN(TabFinder);
 };
 
 #endif  // CHROME_BROWSER_TABS_TAB_FINDER_H_