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

chrome/browser/extensions/api/bluetooth/bluetooth_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_BLUETOOTH_BLUETOOTH_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_BLUETOOTH_BLUETOOTH_API_H_
 #pragma once
 
 #include "chrome/browser/extensions/extension_function.h"
 
 #if defined(OS_CHROMEOS)
 namespace chromeos {
 
 class BluetoothAdapter;
 class BluetoothDevice;
 
 }  // namespace chromeos
 #endif
 
 namespace extensions {
 namespace api {
 
 class BluetoothExtensionFunction : public SyncExtensionFunction {
  protected:
+  virtual ~BluetoothExtensionFunction() {}
+
 #if defined(OS_CHROMEOS)
   const chromeos::BluetoothAdapter* adapter() const;
   chromeos::BluetoothAdapter* GetMutableAdapter();
 #endif
 };
 
 class AsyncBluetoothExtensionFunction : public AsyncExtensionFunction {
  protected:
 #if defined(OS_CHROMEOS)
   const chromeos::BluetoothAdapter* adapter() const;
   chromeos::BluetoothAdapter* GetMutableAdapter();
 #endif
 };
 
 class BluetoothIsAvailableFunction : public BluetoothExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.isAvailable")
+
+ protected:
+  virtual ~BluetoothIsAvailableFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.isAvailable")
 };
 
 class BluetoothIsPoweredFunction : public BluetoothExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.isPowered")
+
+ protected:
+  virtual ~BluetoothIsPoweredFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.isPowered")
 };
 
 class BluetoothGetAddressFunction : public BluetoothExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.getAddress")
+
+ protected:
+  virtual ~BluetoothGetAddressFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.getAddress")
 };
 
 class BluetoothGetDevicesWithServiceUUIDFunction
     : public BluetoothExtensionFunction {
  public:
-  virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION_NAME(
       "experimental.bluetooth.getDevicesWithServiceUUID")
+
+ protected:
+  virtual ~BluetoothGetDevicesWithServiceFunction() {}
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
 };
 
 class BluetoothGetDevicesWithServiceNameFunction
     : public AsyncBluetoothExtensionFunction {
  public:
-  virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION_NAME(
       "experimental.bluetooth.getDevicesWithServiceName")
 
+ protected:
 #if defined(OS_CHROMEOS)
   BluetoothGetDevicesWithServiceNameFunction();
+  virtual ~BluetoothGetDevicesWithServiceNameFunction();
+#endif
 
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
+
+#if defined(OS_CHROMEOS)
  private:
   void AddDeviceIfTrue(
       ListValue* list, const chromeos::BluetoothDevice* device, bool result);
 
   int callbacks_pending_;
 #endif
 };
 
 class BluetoothDisconnectFunction : public BluetoothExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.disconnect")
+
+ protected:
+  virtual ~BluetoothDisconnectFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.disconnect")
 };
 
 class BluetoothReadFunction : public BluetoothExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.read")
+
+ protected:
+  virtual ~BluetoothReadFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.read")
 };
 
 class BluetoothSetOutOfBandPairingDataFunction
     : public BluetoothExtensionFunction {
  public:
-  virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION_NAME(
       "experimental.bluetooth.setOutOfBandPairingData")
+
+ protected:
+  virtual ~BluetoothSetOutOfBandPairingDataFunction() {}
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
 };
 
 class BluetoothGetOutOfBandPairingDataFunction
     : public BluetoothExtensionFunction {
  public:
-  virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION_NAME(
       "experimental.bluetooth.getOutOfBandPairingData")
+
+ protected:
+  virtual ~BluetoothGetOutOfBandPairingDataFunction() {}
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
 };
 
 class BluetoothWriteFunction : public BluetoothExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.write")
+
+ private:
+  virtual ~BluetoothWriteFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.write")
 };
 
 class BluetoothConnectFunction : public BluetoothExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.connect")
+
+ protected:
+  virtual ~BluetoothConnectFunction() {}
+
+  // ExtensionFunction:
   virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.bluetooth.connect")
 };
 
 }  // namespace api
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_BLUETOOTH_BLUETOOTH_API_H_