[Draft] Fix canceling pin prompt causes host overload

remoting/protocol/session.h

Index: remoting/protocol/session.h
diff --git a/remoting/protocol/session.h b/remoting/protocol/session.h
index a542283001e6ced5fbb0359d11895c9a2442d9cd..d370c785ead02b7fc939d7f004680aa006076ac5 100644
--- a/remoting/protocol/session.h
+++ b/remoting/protocol/session.h
@@ -69,6 +69,9 @@ class Session {
     // means and how it can used.
     virtual void OnSessionChannelReady(const std::string& channel_name,
                                        bool ready) {}
+
+    // Called when the session begins authentication
+    virtual void OnSessionAuthenticationBegin() {};

[Sergey Ulanov, 2014/03/24 18:42:36]

Instead of adding this callback I think it would be cleaner to have another
state between CONNECTED and AUTHENTICATED.

[Sergey Ulanov, 2014/03/24 18:42:36]

If you keep this method: s/Begin/Started/ for consistency.

[kelvinp, 2014/03/24 23:11:39]

Done.

[kelvinp, 2014/03/24 23:11:39]

I have added the extra state instead

   };