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

chrome/browser/prefs/value_map_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/prefs/value_map_pref_store.h"
 
 #include <algorithm>
 
 #include "base/stl_util.h"
 #include "base/values.h"
 
 ValueMapPrefStore::ValueMapPrefStore() {}
 
-ValueMapPrefStore::~ValueMapPrefStore() {}
-
 PrefStore::ReadResult ValueMapPrefStore::GetValue(const std::string& key,
                                                   const Value** value) const {
   return prefs_.GetValue(key, value) ? READ_OK : READ_NO_VALUE;
 }
 
 void ValueMapPrefStore::AddObserver(PrefStore::Observer* observer) {
   observers_.AddObserver(observer);
 }
 
 void ValueMapPrefStore::RemoveObserver(PrefStore::Observer* observer) {
   observers_.RemoveObserver(observer);
 }
 
 size_t ValueMapPrefStore::NumberOfObservers() const {
   return observers_.size();
 }
 
-void ValueMapPrefStore::SetValue(const std::string& key, Value* value) {
-  if (prefs_.SetValue(key, value))
-    FOR_EACH_OBSERVER(Observer, observers_, OnPrefValueChanged(key));
-}
-
-void ValueMapPrefStore::RemoveValue(const std::string& key) {
-  if (prefs_.RemoveValue(key))
-    FOR_EACH_OBSERVER(Observer, observers_, OnPrefValueChanged(key));
-}
-
-void ValueMapPrefStore::NotifyInitializationCompleted() {
-  FOR_EACH_OBSERVER(Observer, observers_, OnInitializationCompleted(true));
-}
-
 ValueMapPrefStore::iterator ValueMapPrefStore::begin() {
   return prefs_.begin();
 }
 
 ValueMapPrefStore::iterator ValueMapPrefStore::end() {
   return prefs_.end();
 }
 
 ValueMapPrefStore::const_iterator ValueMapPrefStore::begin() const {
   return prefs_.begin();
 }
 
 ValueMapPrefStore::const_iterator ValueMapPrefStore::end() const {
   return prefs_.end();
 }
+
+ValueMapPrefStore::~ValueMapPrefStore() {}
+
+void ValueMapPrefStore::SetValue(const std::string& key, Value* value) {
+  if (prefs_.SetValue(key, value))
+    FOR_EACH_OBSERVER(Observer, observers_, OnPrefValueChanged(key));
+}
+
+void ValueMapPrefStore::RemoveValue(const std::string& key) {
+  if (prefs_.RemoveValue(key))
+    FOR_EACH_OBSERVER(Observer, observers_, OnPrefValueChanged(key));
+}
+
+void ValueMapPrefStore::NotifyInitializationCompleted() {
+  FOR_EACH_OBSERVER(Observer, observers_, OnInitializationCompleted(true));
+}