Virtual destructors should have virtual keyword.

chrome/browser/policy/configuration_policy_provider.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_POLICY_CONFIGURATION_POLICY_PROVIDER_H_
 #define CHROME_BROWSER_POLICY_CONFIGURATION_POLICY_PROVIDER_H_
 #pragma once
 
 #include <map>
 #include <string>
@@ skipping 85 matching lines @@
 
 // Manages observers for a ConfigurationPolicyProvider. Is used to register
 // observers, and automatically removes them upon destruction.
 // Implementation detail: to avoid duplicate bookkeeping of registered
 // observers, this registrar class acts as a proxy for notifications (since it
 // needs to register itself anyway to get OnProviderGoingAway notifications).
 class ConfigurationPolicyObserverRegistrar
     : ConfigurationPolicyProvider::Observer {
  public:
   ConfigurationPolicyObserverRegistrar();
-  ~ConfigurationPolicyObserverRegistrar();
+  virtual ~ConfigurationPolicyObserverRegistrar();
   void Init(ConfigurationPolicyProvider* provider,
             ConfigurationPolicyProvider::Observer* observer);
 
   // ConfigurationPolicyProvider::Observer implementation:
   virtual void OnUpdatePolicy();
   virtual void OnProviderGoingAway();
 
  private:
   ConfigurationPolicyProvider* provider_;
   ConfigurationPolicyProvider::Observer* observer_;
 
   DISALLOW_COPY_AND_ASSIGN(ConfigurationPolicyObserverRegistrar);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_CONFIGURATION_POLICY_PROVIDER_H_