Virtual destructors should have virtual keyword.

chrome/browser/instant/instant_controller.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_INSTANT_INSTANT_CONTROLLER_H_
 #define CHROME_BROWSER_INSTANT_INSTANT_CONTROLLER_H_
 #pragma once
 
 #include <set>
 #include <string>
@@ skipping 33 matching lines @@
 // by invoking |CommitCurrentPreview|, which results in |CommitInstant|
 // being invoked on the delegate.
 class InstantController : public InstantLoaderDelegate {
  public:
   // Amount of time to wait before starting the instant animation.
   static const int kAutoCommitPauseTimeMS = 1000;
   // Duration of the instant animation in which the colors change.
   static const int kAutoCommitFadeInTimeMS = 300;
 
   InstantController(Profile* profile, InstantDelegate* delegate);
-  ~InstantController();
+  virtual ~InstantController();
 
   // Registers instant related preferences.
   static void RegisterUserPrefs(PrefService* prefs);
 
   // Records instant metrics.
   static void RecordMetrics(Profile* profile);
 
   // Returns true if instant is enabled.
   static bool IsEnabled(Profile* profile);
 
@@ skipping 231 matching lines @@
   ScopedVector<InstantLoader> loaders_to_destroy_;
 
   // The set of hosts that we don't use instant with. This is shared across all
   // instances and only maintained for the current session.
   static HostBlacklist* host_blacklist_;
 
   DISALLOW_COPY_AND_ASSIGN(InstantController);
 };
 
 #endif  // CHROME_BROWSER_INSTANT_INSTANT_CONTROLLER_H_