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

chrome/browser/extensions/extension_input_ime_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_INPUT_IME_API_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_INPUT_IME_API_H_
 #pragma once
 
 #include "chrome/browser/extensions/extension_function.h"
 
@@ skipping 49 matching lines @@
       observers_;
 
   unsigned int next_request_id_;
   RequestMap request_map_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionInputImeEventRouter);
 };
 
 class SetCompositionFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setComposition");
+
+ protected:
   virtual ~SetCompositionFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setComposition");
 };
 
 class ClearCompositionFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.clearComposition");
+
+ protected:
   virtual ~ClearCompositionFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.clearComposition");
 };
 
 class CommitTextFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.commitText");
+
+ protected:
   virtual ~CommitTextFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.commitText");
 };
 
 class SetCandidateWindowPropertiesFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setCandidateWindowProperties");
+
+ protected:
   virtual ~SetCandidateWindowPropertiesFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setCandidateWindowProperties");
 };
 
 class SetCandidatesFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setCandidates");
+
+ protected:
   virtual ~SetCandidatesFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setCandidates");
+
  private:
   bool ReadCandidates(
       ListValue* candidates,
       std::vector<chromeos::InputMethodEngine::Candidate>* output);
 };
 
 class SetCursorPositionFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setCursorPosition");
+
+ protected:
   virtual ~SetCursorPositionFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setCursorPosition");
 };
 
 class SetMenuItemsFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setMenuItems");
+
+ protected:
   virtual ~SetMenuItemsFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.setMenuItems");
 };
 
 class UpdateMenuItemsFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.updateMenuItems");
+
+ protected:
   virtual ~UpdateMenuItemsFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.updateMenuItems");
 };
 
 class InputEventHandled : public AsyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.eventHandled");
+
+ protected:
   virtual ~InputEventHandled() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("input.ime.eventHandled");
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_INPUT_IME_API_H_