[Draft] Fix canceling pin prompt causes host overload

remoting/protocol/session.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 REMOTING_PROTOCOL_SESSION_H_
 #define REMOTING_PROTOCOL_SESSION_H_
 
 #include <string>
 
 #include "remoting/protocol/errors.h"
@@ skipping 20 matching lines @@
 
     // Sent session-initiate, but haven't received session-accept.
     CONNECTING,
 
     // Received session-initiate, but haven't sent session-accept.
     ACCEPTING,
 
     // Session has been accepted and is pending authentication.
     CONNECTED,
 
+    // Session has started authenticating.
+    AUTHENTICATING,
+
     // Session has been connected and authenticated.
     AUTHENTICATED,
 
     // Session has been closed.
     CLOSED,
 
     // Connection has failed.
     FAILED,
   };
 
   class EventHandler {
    public:
     EventHandler() {}
     virtual ~EventHandler() {}
 
     // Called after session state has changed. It is safe to destroy
-    // the session from within the handler if |state| is CLOSED or
-    // FAILED.
+    // the session from within the handler if |state| is AUTHENTICATING
+    // or CLOSED or FAILED.
     virtual void OnSessionStateChange(State state) = 0;
 
     // Called whenever route for the channel specified with
     // |channel_name| changes. Session must not be destroyed by the
     // handler of this event.
     virtual void OnSessionRouteChange(const std::string& channel_name,
                                       const TransportRoute& route) = 0;
 
     // Called when ready state on one of the channels changes. See
     // comments in transport.h for explanation on what this state
@@ skipping 42 matching lines @@
   virtual void Close() = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(Session);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_SESSION_H_