RefCounted types should not have public destructors, chrome/browser/ part 5

chrome/browser/policy/configuration_policy_pref_store.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/policy/configuration_policy_pref_store.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 30 matching lines @@
 ConfigurationPolicyPrefStore::ConfigurationPolicyPrefStore(
     PolicyService* service,
     PolicyLevel level)
     : policy_service_(service),
       level_(level) {
   // Read initial policy.
   prefs_.reset(CreatePreferencesFromPolicies());
   policy_service_->AddObserver(POLICY_DOMAIN_CHROME, "", this);
 }
 
-ConfigurationPolicyPrefStore::~ConfigurationPolicyPrefStore() {
-  policy_service_->RemoveObserver(POLICY_DOMAIN_CHROME, "", this);
-}
-
 void ConfigurationPolicyPrefStore::AddObserver(PrefStore::Observer* observer) {
   observers_.AddObserver(observer);
 }
 
 void ConfigurationPolicyPrefStore::RemoveObserver(
     PrefStore::Observer* observer) {
   observers_.RemoveObserver(observer);
 }
 
 size_t ConfigurationPolicyPrefStore::NumberOfObservers() const {
@@ skipping 38 matching lines @@
                                           POLICY_LEVEL_MANDATORY);
 }
 
 // static
 ConfigurationPolicyPrefStore*
 ConfigurationPolicyPrefStore::CreateRecommendedPolicyPrefStore() {
   return new ConfigurationPolicyPrefStore(g_browser_process->policy_service(),
                                           POLICY_LEVEL_RECOMMENDED);
 }
 
+ConfigurationPolicyPrefStore::~ConfigurationPolicyPrefStore() {
+  policy_service_->RemoveObserver(POLICY_DOMAIN_CHROME, "", this);
+}
+
 void ConfigurationPolicyPrefStore::Refresh() {
   scoped_ptr<PrefValueMap> new_prefs(CreatePreferencesFromPolicies());
   std::vector<std::string> changed_prefs;
   new_prefs->GetDifferingKeys(prefs_.get(), &changed_prefs);
   prefs_.swap(new_prefs);
 
   // Send out change notifications.
   for (std::vector<std::string>::const_iterator pref(changed_prefs.begin());
        pref != changed_prefs.end();
        ++pref) {
@@ skipping 24 matching lines @@
   // issue during startup.
   BrowserThread::PostTask(BrowserThread::UI,
                           FROM_HERE,
                           base::Bind(&LogErrors,
                                      base::Owned(errors.release())));
 
   return prefs.release();
 }
 
 }  // namespace policy