RefCounted types should not have public destructors, chrome/browser/ part 1

chrome/browser/history/history_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_HISTORY_HISTORY_EXTENSION_API_H_
 #define CHROME_BROWSER_HISTORY_HISTORY_EXTENSION_API_H_
 #pragma once
 
 #include <string>
 
@@ skipping 30 matching lines @@
 
   // Used for tracking registrations to history service notifications.
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(HistoryExtensionEventRouter);
 };
 
 
 // Base class for history function APIs.
 class HistoryFunction : public AsyncExtensionFunction {
- public:
+ protected:
+  virtual ~HistoryFunction() {}
   virtual void Run() OVERRIDE;
-  virtual bool RunImpl() = 0;
 
   bool GetUrlFromValue(base::Value* value, GURL* url);
   bool GetTimeFromValue(base::Value* value, base::Time* time);
 };
 
 // Base class for history funciton APIs which require async interaction with
 // chrome services and the extension thread.
 class HistoryFunctionWithCallback : public HistoryFunction {
  public:
   HistoryFunctionWithCallback();
+
+ protected:
   virtual ~HistoryFunctionWithCallback();
 
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
+
   // Return true if the async call was completed, false otherwise.
   virtual bool RunAsyncImpl() = 0;
 
   // Call this method to report the results of the async method to the caller.
   // This method calls Release().
   virtual void SendAsyncResponse();
 
-  // Override HistoryFunction::RunImpl.
-  virtual bool RunImpl() OVERRIDE;
-
- protected:
   // The consumer for the HistoryService callbacks.
   CancelableRequestConsumer cancelable_consumer_;
 
  private:
   // The actual call to SendResponse.  This is required since the semantics for
   // CancelableRequestConsumerT require it to be accessed after the call.
   void SendResponseToCallback();
 };
 
 class GetVisitsHistoryFunction : public HistoryFunctionWithCallback {
  public:
-  // Override HistoryFunction.
+  DECLARE_EXTENSION_FUNCTION_NAME("history.getVisits");
+
+ protected:
+  virtual ~GetVisitsHistoryFunction() {}
+
+  // HistoryFunctionWithCallback:
   virtual bool RunAsyncImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("history.getVisits");
 
   // Callback for the history function to provide results.
   void QueryComplete(HistoryService::Handle request_service,
                      bool success,
                      const history::URLRow* url_row,
                      history::VisitVector* visits);
 };
 
 class SearchHistoryFunction : public HistoryFunctionWithCallback {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("history.search");
+
+ protected:
+  virtual ~SearchHistoryFunction() {}
+
+  // HistoryFunctionWithCallback:
   virtual bool RunAsyncImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("history.search");
 
   // Callback for the history function to provide results.
   void SearchComplete(HistoryService::Handle request_handle,
                       history::QueryResults* results);
 };
 
 class AddUrlHistoryFunction : public HistoryFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("history.addUrl");
+
+ protected:
+  virtual ~AddUrlHistoryFunction() {}
+
+  // HistoryFunctionWithCallback:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("history.addUrl");
 };
 
 class DeleteAllHistoryFunction : public HistoryFunctionWithCallback {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("history.deleteAll");
+
+ protected:
+  virtual ~DeleteAllHistoryFunction() {}
+
+  // HistoryFunctionWithCallback:
   virtual bool RunAsyncImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("history.deleteAll");
 
   // Callback for the history service to acknowledge deletion.
   void DeleteComplete();
 };
 
 
 class DeleteUrlHistoryFunction : public HistoryFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("history.deleteUrl");
+
+ protected:
+  virtual ~DeleteUrlHistoryFunction() {}
+
+  // HistoryFunctionWithCallback:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("history.deleteUrl");
 };
 
 class DeleteRangeHistoryFunction : public HistoryFunctionWithCallback {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("history.deleteRange");
+
+ protected:
+  virtual ~DeleteRangeHistoryFunction() {}
+
+  // HistoryFunctionWithCallback:
   virtual bool RunAsyncImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("history.deleteRange");
 
   // Callback for the history service to acknowledge deletion.
   void DeleteComplete();
 };
 
 #endif  // CHROME_BROWSER_HISTORY_HISTORY_EXTENSION_API_H_