Virtual destructors should have virtual keyword.

chrome/browser/policy/profile_policy_connector.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_PROFILE_POLICY_CONNECTOR_H_
 #define CHROME_BROWSER_POLICY_PROFILE_POLICY_CONNECTOR_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "chrome/browser/policy/configuration_policy_provider.h"
 #include "chrome/browser/profiles/profile_keyed_service.h"
 
 class Profile;
 
 namespace policy {
 
 class CloudPolicySubsystem;
 class UserPolicyIdentityStrategy;
 
 // This class is a container for the profile-specific policy bits located in the
 // profile. Since the subsystem owns the policy provider, it's vital that it
 // gets initialized before the profile's prefs and destroyed after the prefs
 // are gone.
 class ProfilePolicyConnector : public ProfileKeyedService {
  public:
   explicit ProfilePolicyConnector(Profile* profile);
-  ~ProfilePolicyConnector();
+  virtual ~ProfilePolicyConnector();
 
   // Schedules initialization of the policy backend service if the service is
   // already constructed.
   void ScheduleServiceInitialization(int delay_milliseconds);
 
   // Initializes the context. Should be called only after the profile's request
   // context is up.
   void Initialize();
 
   // Shuts the context down. This must be called before the networking
@@ skipping 53 matching lines @@
   scoped_ptr<ConfigurationPolicyObserverRegistrar> profile_registrar_;
   ObserverList<ConfigurationPolicyProvider::Observer, true> observer_list_;
 
   DISALLOW_COPY_AND_ASSIGN(MergingPolicyProvider);
 };
 
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_PROFILE_POLICY_CONNECTOR_H_