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

chrome/browser/extensions/api/extension_action/extension_browser_actions_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_API_EXTENSION_ACTION_EXTENSION_BROWSER_ACTIONS_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_EXTENSION_ACTION_EXTENSION_BROWSER_ACTIONS_API_H_
 #pragma once
 
 #include "chrome/browser/extensions/api/extension_action/extension_actions_api.h"
 #include "chrome/browser/extensions/extension_function.h"
 #include "chrome/common/extensions/extension_action.h"
 
 // Base class for chrome.browserAction.* APIs.
 class BrowserActionFunction : public ExtensionActionFunction {
  protected:
   virtual ~BrowserActionFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
   void FireUpdateNotification();
 };
 
 // Implement chrome.browserAction.setIcon().
 class BrowserActionSetIconFunction : public BrowserActionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setIcon")
+
+ protected:
   virtual ~BrowserActionSetIconFunction() {}
+
+  // ExtensionActionFunction:
   virtual bool RunExtensionAction() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setIcon")
 };
 
 // Implement chrome.browserAction.setTitle().
 class BrowserActionSetTitleFunction : public BrowserActionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setTitle")
+
+ protected:
   virtual ~BrowserActionSetTitleFunction() {}
+
+  // ExtensionActionFunction:
   virtual bool RunExtensionAction() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setTitle")
 };
 
 // Implement chrome.browserAction.setPopup().
 class BrowserActionSetPopupFunction : public BrowserActionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setPopup")
+
+ protected:
   virtual ~BrowserActionSetPopupFunction() {}
+
+  // ExtensionActionFunction:
   virtual bool RunExtensionAction() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setPopup")
 };
 
 // Implement chrome.browserAction.setBadgeText().
 class BrowserActionSetBadgeTextFunction : public BrowserActionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setBadgeText")
+
+ protected:
   virtual ~BrowserActionSetBadgeTextFunction() {}
+
+  // ExtensionActionFunction:
   virtual bool RunExtensionAction() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setBadgeText")
 };
 
 // Implement chrome.browserAction.setBadgeBackgroundColor().
 class BrowserActionSetBadgeBackgroundColorFunction
     : public BrowserActionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setBadgeBackgroundColor")
+
+ protected:
   virtual ~BrowserActionSetBadgeBackgroundColorFunction() {}
+
+  // ExtensionActionFunction:
   virtual bool RunExtensionAction() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.setBadgeBackgroundColor")
 };
 
 // Implement chrome.browserAction.getTitle().
 class BrowserActionGetTitleFunction : public BrowserActionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.getTitle")
+
+ protected:
   virtual ~BrowserActionGetTitleFunction() {}
+
+  // ExtensionActionFunction:
   virtual bool RunExtensionAction() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.getTitle")
 };
 
 // Implement chrome.browserAction.getPopup().
 class BrowserActionGetPopupFunction : public BrowserActionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.getPopup")
+
+ protected:
   virtual ~BrowserActionGetPopupFunction() {}
+
+  // ExtensionActionFunction:
   virtual bool RunExtensionAction() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.getPopup")
 };
 
 // Implement chrome.browserAction.getBadgeText().
 class BrowserActionGetBadgeTextFunction : public BrowserActionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.getBadgeText")
+
+ protected:
   virtual ~BrowserActionGetBadgeTextFunction() {}
+
+  // ExtensionActionFunction:
   virtual bool RunExtensionAction() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.getBadgeText")
 };
 
 // Implement chrome.browserAction.getBadgeBackgroundColor().
 class BrowserActionGetBadgeBackgroundColorFunction
     : public BrowserActionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.getBadgeBackgroundColor")
+
+ protected:
   virtual ~BrowserActionGetBadgeBackgroundColorFunction() {}
+
+  // ExtensionActionFunction:
   virtual bool RunExtensionAction() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("browserAction.getBadgeBackgroundColor")
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_EXTENSION_ACTION_EXTENSION_BROWSER_ACTIONS_API_H_