RefCounted types should not have public destructors, chrome/browser/bookmarks/ edition

chrome/browser/bookmarks/bookmark_manager_extension_api.h

 // Copyright (c) 2011 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_BOOKMARKS_BOOKMARK_MANAGER_EXTENSION_API_H_
 #define CHROME_BROWSER_BOOKMARKS_BOOKMARK_MANAGER_EXTENSION_API_H_
 #pragma once
 
 #include "chrome/browser/bookmarks/bookmark_extension_api.h"
 #include "chrome/browser/bookmarks/bookmark_node_data.h"
@@ skipping 37 matching lines @@
 
   Profile* profile_;
   TabContentsWrapper* tab_;
   BookmarkNodeData bookmark_drag_data_;
 
   DISALLOW_COPY_AND_ASSIGN(BookmarkManagerExtensionEventRouter);
 };
 
 class ClipboardBookmarkManagerFunction : public BookmarksFunction {
  protected:
+  virtual ~ClipboardBookmarkManagerFunction() {}
+
   bool CopyOrCut(bool cut);
 };
 
 class CopyBookmarkManagerFunction : public ClipboardBookmarkManagerFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.copy");
+
+ protected:
+  virtual ~CopyBookmarkManagerFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.copy");
 };
 
 class CutBookmarkManagerFunction : public ClipboardBookmarkManagerFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.cut");
+
+ protected:
+  virtual ~CutBookmarkManagerFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.cut");
 };
 
 class PasteBookmarkManagerFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.paste");
+
+ protected:
+  virtual ~PasteBookmarkManagerFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.paste");
 };
 
 class CanPasteBookmarkManagerFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.canPaste");
+
+ protected:
+  virtual ~CanPasteBookmarkManagerFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.canPaste");
 };
 
 class SortChildrenBookmarkManagerFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.sortChildren");
+
+ protected:
+  virtual ~SortChildrenBookmarkManagerFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.sortChildren");
 };
 
 class BookmarkManagerGetStringsFunction : public AsyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.getStrings");
+
+ protected:
+  virtual ~BookmarkManagerGetStringsFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.getStrings");
 };
 
 class StartDragBookmarkManagerFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.startDrag");
+
+ protected:
+  virtual ~StartDragBookmarkManagerFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.startDrag");
 };
 
 class DropBookmarkManagerFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.drop");
+
+ protected:
+  virtual ~DropBookmarkManagerFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.drop");
 };
 
 class GetSubtreeBookmarkManagerFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.getSubtree");
+
+ protected:
+  virtual ~GetSubtreeBookmarkManagerFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.getSubtree");
 };
 
 class CanEditBookmarkManagerFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.canEdit");
+
+ protected:
+  virtual ~CanEditBookmarkManagerFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.canEdit");
 };
 
 class RecordLaunchBookmarkFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.recordLaunch")
+
+ protected:
+  virtual ~RecordLaunchBookmarkFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bookmarkManager.recordLaunch")
 };
 
 #endif  // CHROME_BROWSER_BOOKMARKS_BOOKMARK_MANAGER_EXTENSION_API_H_