RefCounted types should not have public destructors, delegate cleanup

remoting/protocol/session_manager.h

Index: remoting/protocol/session_manager.h
diff --git a/remoting/protocol/session_manager.h b/remoting/protocol/session_manager.h
index eb0c9db13b1e549be7d19280d03128811c592d50..fc9645c1a5f869491e45ffb10bd87a872e7d2e36 100644
--- a/remoting/protocol/session_manager.h
+++ b/remoting/protocol/session_manager.h
@@ -76,8 +76,8 @@ class AuthenticatorFactory;
 // client side and one for the host side.
 class SessionManager : public base::NonThreadSafe {
  public:
-  SessionManager() { }
-  virtual ~SessionManager() { }
+  SessionManager() {}
+  virtual ~SessionManager() {}
 
   enum IncomingSessionResponse {
     // Accept the session.
@@ -97,8 +97,7 @@ class SessionManager : public base::NonThreadSafe {
 
   class Listener {
    public:
-    Listener() { }
-    ~Listener() { }
+    Listener() {}
 
     // Called when the session manager is ready to create outgoing
     // sessions. May be called from Init() or after Init()
@@ -115,6 +114,9 @@ class SessionManager : public base::NonThreadSafe {
     // The callback must take ownership of the |session| if it ACCEPTs it.
     virtual void OnIncomingSession(Session* session,
                                    IncomingSessionResponse* response) = 0;
+
+   protected:
+    ~Listener() {}

[Sergey Ulanov, 2012/05/21 19:13:50]

add virtual please

   };
 
   // Initializes the session client. Caller retains ownership of the