RefCounted types should not have public destructors, chrome/browser/extensions

chrome/browser/extensions/extension_preference_api.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_PREFERENCE_API_H__
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_PREFERENCE_API_H__
 #pragma once
 
 #include <string>
 
@@ skipping 53 matching lines @@
   // into a representation that is used by the extension.
   // Returns the extension representation in case of success or NULL otherwise.
   // The ownership of the returned value is passed to the caller.
   virtual base::Value* BrowserToExtensionPref(
       const base::Value* browser_pref) = 0;
 };
 
 // A base class to provide functionality common to the other *PreferenceFunction
 // classes.
 class PreferenceFunction : public SyncExtensionFunction {
- public:
+ protected:
   virtual ~PreferenceFunction();
 
- protected:
   // Given an |extension_pref_key|, provides its |browser_pref_key| from the
   // static map in extension_preference.cc. Returns true if the corresponding
   // browser pref exists and the extension has the API permission needed to
   // modify that pref. Sets |error_| if the extension doesn't have the needed
   // permission.
   bool ValidateBrowserPref(const std::string& extension_pref_key,
                            std::string* browser_pref_key);
 };
 
 class GetPreferenceFunction : public PreferenceFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("types.ChromeSetting.get")
+
+ protected:
   virtual ~GetPreferenceFunction();
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("types.ChromeSetting.get")
 };
 
 class SetPreferenceFunction : public PreferenceFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("types.ChromeSetting.set")
+
+ protected:
   virtual ~SetPreferenceFunction();
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("types.ChromeSetting.set")
 };
 
 class ClearPreferenceFunction : public PreferenceFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("types.ChromeSetting.clear")
+
+ protected:
   virtual ~ClearPreferenceFunction();
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("types.ChromeSetting.clear")
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_PREFERENCE_API_H__