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

chrome/browser/extensions/extension_web_socket_proxy_private_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_EXTENSIONS_EXTENSION_WEB_SOCKET_PROXY_PRIVATE_API_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_WEB_SOCKET_PROXY_PRIVATE_API_H_
 #pragma once
 
 #include "base/timer.h"
 #include "chrome/browser/extensions/extension_function.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "net/base/address_list.h"
 
 class IOThread;
 
 namespace net {
 class SingleRequestHostResolver;
 }
 
 // Base class for web socket proxy functions.
 class WebSocketProxyPrivate
     : public AsyncExtensionFunction, public content::NotificationObserver {
  public:
   WebSocketProxyPrivate();
+
+ protected:
   virtual ~WebSocketProxyPrivate();
 
- protected:
   // Custom finalization.
   virtual void CustomFinalize() = 0;
 
   // ExtensionFunction implementation.
   virtual bool RunImpl() OVERRIDE;
 
   // content::NotificationObserver implementation.
   virtual void Observe(
       int type, const content::NotificationSource& source,
       const content::NotificationDetails& details) OVERRIDE;
@@ skipping 31 matching lines @@
   // Used to cancel host resolution when out of scope.
   scoped_ptr<net::SingleRequestHostResolver> resolver_;
   // Callback which is called when host is resolved.
   bool is_finalized_;
 };
 
 // New API function for web socket proxy, which should be used.
 class WebSocketProxyPrivateGetURLForTCPFunction
     : public WebSocketProxyPrivate {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("webSocketProxyPrivate.getURLForTCP")
+
   WebSocketProxyPrivateGetURLForTCPFunction();
+
+ protected:
   virtual ~WebSocketProxyPrivateGetURLForTCPFunction();
 
- private:
   // ExtensionFunction implementation.
   virtual bool RunImpl() OVERRIDE;
 
   // WebSocketProxyPrivate implementation:
   virtual void CustomFinalize() OVERRIDE;
-
-  DECLARE_EXTENSION_FUNCTION_NAME("webSocketProxyPrivate.getURLForTCP")
 };
 
 // Legacy API function for web socket proxy, to be eliminated.
 class WebSocketProxyPrivateGetPassportForTCPFunction
     : public WebSocketProxyPrivate {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("webSocketProxyPrivate.getPassportForTCP")
+
   WebSocketProxyPrivateGetPassportForTCPFunction();
+
+ protected:
   virtual ~WebSocketProxyPrivateGetPassportForTCPFunction();
 
- private:
   // WebSocketProxyPrivate implementation:
   virtual void CustomFinalize() OVERRIDE;
-
-  DECLARE_EXTENSION_FUNCTION_NAME("webSocketProxyPrivate.getPassportForTCP")
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_WEB_SOCKET_PROXY_PRIVATE_API_H_