Virtual destructors should have virtual keyword.

chrome/browser/intranet_redirect_detector.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_INTRANET_REDIRECT_DETECTOR_H_
 #define CHROME_BROWSER_INTRANET_REDIRECT_DETECTOR_H_
 #pragma once
 
 #include <set>
 #include <string>
@@ skipping 28 matching lines @@
     : public URLFetcher::Delegate,
       public net::NetworkChangeNotifier::IPAddressObserver {
  public:
   // Only the main browser process loop should call this, when setting up
   // g_browser_process->intranet_redirect_detector_.  No code other than the
   // IntranetRedirectDetector itself should actually use
   // g_browser_process->intranet_redirect_detector() (which shouldn't be hard,
   // since there aren't useful public functions on this object for consumers to
   // access anyway).
   IntranetRedirectDetector();
-  ~IntranetRedirectDetector();
+  virtual ~IntranetRedirectDetector();
 
   // Returns the current redirect origin.  This will be empty if no redirection
   // is in place.
   static GURL RedirectOrigin();
 
   static void RegisterPrefs(PrefService* prefs);
 
   // The number of characters the fetcher will use for its randomly-generated
   // hostnames.
   static const size_t kNumCharsInHostnames;
@@ skipping 34 matching lines @@
   explicit IntranetRedirectHostResolverProc(net::HostResolverProc* previous);
 
   virtual int Resolve(const std::string& host,
                       net::AddressFamily address_family,
                       net::HostResolverFlags host_resolver_flags,
                       net::AddressList* addrlist,
                       int* os_error);
 };
 
 #endif  // CHROME_BROWSER_INTRANET_REDIRECT_DETECTOR_H_