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

chrome/browser/extensions/extension_context_menu_api.h

Index: chrome/browser/extensions/extension_context_menu_api.h
diff --git a/chrome/browser/extensions/extension_context_menu_api.h b/chrome/browser/extensions/extension_context_menu_api.h
index dc6e4d9755f3d7eafd0a886dfafbf577f02fd17e..840ac5a9e546d883de8073ec7838c3ce464b24f5 100644
--- a/chrome/browser/extensions/extension_context_menu_api.h
+++ b/chrome/browser/extensions/extension_context_menu_api.h
@@ -17,10 +17,9 @@ 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,
@@ -63,27 +62,47 @@ class ExtensionContextMenuFunction : public SyncExtensionFunction {
 };
 
 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__