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

chrome/browser/bookmarks/bookmark_extension_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_BOOKMARKS_BOOKMARK_EXTENSION_API_H_
 #define CHROME_BROWSER_BOOKMARKS_BOOKMARK_EXTENSION_API_H_
 #pragma once
 
 #include <list>
 #include <string>
@@ skipping 56 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(BookmarkExtensionEventRouter);
 };
 
 class BookmarksFunction : public AsyncExtensionFunction,
                           public content::NotificationObserver {
  public:
   // AsyncExtensionFunction:
   virtual void Run() OVERRIDE;
 
-  virtual bool RunImpl() = 0;
+ protected:
+  virtual ~BookmarksFunction() {}
 
- protected:
   // Helper to get the bookmark id as int64 from the given string id.
   // Sets error_ to an error string if the given id string can't be parsed
   // as an int64. In case of error, doesn't change id and returns false.
   bool GetBookmarkIdAsInt64(const std::string& id_string, int64* id);
 
   // Helper that checks if bookmark editing is enabled. If it's not, this sets
   // error_ to the appropriate error string.
   bool EditBookmarksEnabled();
 
  private:
   // content::NotificationObserver:
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   content::NotificationRegistrar registrar_;
 };
 
 class GetBookmarksFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.get")
+
+ protected:
+  virtual ~GetBookmarksFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.get")
 };
 
 class GetBookmarkChildrenFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.getChildren")
+
+ protected:
+  virtual ~GetBookmarkChildrenFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.getChildren")
 };
 
 class GetBookmarkRecentFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.getRecent")
+
+ protected:
+  virtual ~GetBookmarkRecentFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.getRecent")
 };
 
 class GetBookmarkTreeFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.getTree")
+
+ protected:
+  virtual ~GetBookmarkTreeFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.getTree")
 };
 
 class GetBookmarkSubTreeFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.getSubTree")
+
+ protected:
+  virtual ~GetBookmarkSubTreeFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.getSubTree")
 };
 
 class SearchBookmarksFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.search")
+
+ protected:
+  virtual ~SearchBookmarksFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.search")
 };
 
 class RemoveBookmarkFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.remove")
+
   // Returns true on successful parse and sets invalid_id to true if conversion
   // from id string to int64 failed.
   static bool ExtractIds(const base::ListValue* args, std::list<int64>* ids,
                          bool* invalid_id);
-  // BookmarksFunction:
-  virtual bool RunImpl() OVERRIDE;
+  // ExtensionFunction:
   virtual void GetQuotaLimitHeuristics(
       QuotaLimitHeuristics* heuristics) const OVERRIDE;
 
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.remove")
+ protected:
+  virtual ~RemoveBookmarkFunction() {}
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
 };
 
 class RemoveTreeBookmarkFunction : public RemoveBookmarkFunction {
+ public:
   DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.removeTree")
+
+ protected:
+  virtual ~RemoveTreeBookmarkFunction() {}
 };
 
 class CreateBookmarkFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.create")
+
+  // ExtensionFunction:
   virtual void GetQuotaLimitHeuristics(
       QuotaLimitHeuristics* heuristics) const OVERRIDE;
-  // BookmarksFunction:
+
+ protected:
+  virtual ~CreateBookmarkFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.create")
 };
 
 class MoveBookmarkFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.move")
+
   static bool ExtractIds(const base::ListValue* args, std::list<int64>* ids,
                          bool* invalid_id);
+
+  // ExtensionFunction:
   virtual void GetQuotaLimitHeuristics(
       QuotaLimitHeuristics* heuristics) const OVERRIDE;
-  // BookmarksFunction:
+
+ protected:
+  virtual ~MoveBookmarkFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.move")
 };
 
 class UpdateBookmarkFunction : public BookmarksFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.update")
+
   static bool ExtractIds(const base::ListValue* args, std::list<int64>* ids,
                          bool* invalid_id);
+
+  // ExtensionFunction:
   virtual void GetQuotaLimitHeuristics(
       QuotaLimitHeuristics* heuristics) const OVERRIDE;
+
+ protected:
+  virtual ~UpdateBookmarkFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
- private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.update")
 };
 
 class BookmarksIOFunction : public BookmarksFunction,
                             public SelectFileDialog::Listener {
  public:
   BookmarksIOFunction();
-  virtual ~BookmarksIOFunction();
 
   virtual void FileSelected(const FilePath& path, int index, void* params) = 0;
 
   // SelectFileDialog::Listener:
   virtual void MultiFilesSelected(const std::vector<FilePath>& files,
                                   void* params) OVERRIDE;
   virtual void FileSelectionCanceled(void* params) OVERRIDE;
 
   void SelectFile(SelectFileDialog::Type type);
 
+ protected:
+  virtual ~BookmarksIOFunction();
+
  private:
   void ShowSelectFileDialog(
       SelectFileDialog::Type type,
       const FilePath& default_path);
 
  protected:
   scoped_refptr<SelectFileDialog> select_file_dialog_;
 };
 
 class ImportBookmarksFunction : public BookmarksIOFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.import");
+
   // BookmarkManagerIOFunction:
-  virtual bool RunImpl() OVERRIDE;
   virtual void FileSelected(const FilePath& path, int index, void* params)
       OVERRIDE;
 
  private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.import");
+  virtual ~ImportBookmarksFunction() {}
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
 };
 
 class ExportBookmarksFunction : public BookmarksIOFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.export");
+
   // BookmarkManagerIOFunction:
-  virtual bool RunImpl() OVERRIDE;
   virtual void FileSelected(const FilePath& path, int index, void* params)
       OVERRIDE;
 
  private:
-  DECLARE_EXTENSION_FUNCTION_NAME("bookmarks.export");
+  virtual ~ExportBookmarksFunction() {}
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
 };
 
 #endif  // CHROME_BROWSER_BOOKMARKS_BOOKMARK_EXTENSION_API_H_