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

chrome/browser/extensions/extension_context_menu_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_CONTEXT_MENU_API_H__
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_CONTEXT_MENU_API_H__
 #pragma once
 
 #include "chrome/browser/extensions/extension_function.h"
 #include "chrome/browser/extensions/extension_menu_manager.h"
 #include "chrome/common/extensions/url_pattern_set.h"
 
 class ExtensionMenuItem;
 
 namespace base {
 class DictionaryValue;
 }
 
 class ExtensionContextMenuFunction : public SyncExtensionFunction {
- public:
+ protected:
   virtual ~ExtensionContextMenuFunction() {}
 
- protected:
   // Helper function to read and parse a list of menu item contexts.
   bool ParseContexts(const base::DictionaryValue& properties,
                      const char* key,
                      ExtensionMenuItem::ContextList* result);
 
   // Looks in properties for the "type" key, and reads the value in |result|. On
   // error, returns false and puts an error message into error_. If the key is
   // not present, |result| is set to |default_value| and the return value is
   // true.
   bool ParseType(const base::DictionaryValue& properties,
@@ skipping 22 matching lines @@
   // on error, with an explanation written into error_. Note that if the
   // parentId key is not in properties, this will return true and leave |result|
   // unset. Also, it is considered an error if the item found has a type other
   // than NORMAL.
   bool GetParent(const base::DictionaryValue& properties,
                  const ExtensionMenuManager& manager,
                  ExtensionMenuItem** result);
 };
 
 class CreateContextMenuFunction : public ExtensionContextMenuFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("contextMenus.create")
+
+ protected:
   virtual ~CreateContextMenuFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("contextMenus.create")
 };
 
 class UpdateContextMenuFunction : public ExtensionContextMenuFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("contextMenus.update")
+
+ protected:
   virtual ~UpdateContextMenuFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("contextMenus.update")
 };
 
 class RemoveContextMenuFunction : public ExtensionContextMenuFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("contextMenus.remove")
+
+ protected:
   virtual ~RemoveContextMenuFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("contextMenus.remove")
 };
 
 class RemoveAllContextMenusFunction : public ExtensionContextMenuFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("contextMenus.removeAll")
+
+ protected:
   virtual ~RemoveAllContextMenusFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("contextMenus.removeAll")
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_CONTEXT_MENU_API_H__