Disconnect all users if too many connection requests are received for It2Me

remoting/protocol/authenticator.h

Index: remoting/protocol/authenticator.h
diff --git a/remoting/protocol/authenticator.h b/remoting/protocol/authenticator.h
index 93ecfd72a3129a01da4ff243f50ac9963b274cc9..8add74cd87066b6af30dcafd93da9b5da919a4be 100644
--- a/remoting/protocol/authenticator.h
+++ b/remoting/protocol/authenticator.h
@@ -53,7 +53,7 @@ class Authenticator {
     // Next message is ready to be sent to the peer.
     MESSAGE_READY,
 
-    // Session is authenticated successufully.
+    // Session is authenticated successfully.
     ACCEPTED,
 
     // Session is rejected.
@@ -68,6 +68,7 @@ class Authenticator {
     INVALID_ACCOUNT,
     PROTOCOL_ERROR,
     REJECTED_BY_USER,
+    TOO_MANY_CONNECTIONS,

[Sergey Ulanov, 2017/03/15 22:25:33]

Maybe add comments for each rejection reason? They were not necessary when this
enum had just 2 values (INVALID_CREDENTIALS and PROTOCOL_ERROR), but that's not
the case anymore, and it's more ambiguous now.

[joedow, 2017/03/16 21:32:18]

Done.

   };
 
   // Callback used for layered Authenticator implementations, particularly
@@ -134,7 +135,7 @@ class AuthenticatorFactory {
   // authenticator for the new session. |first_message| specifies
   // authentication part of the session-initiate stanza so that
   // appropriate type of Authenticator can be chosen for the session
-  // (useful when multiple authenticators is supported). Returns nullptr
+  // (useful when multiple authenticators are supported). Returns nullptr
   // if the |first_message| is invalid and the session should be
   // rejected. ProcessMessage() should be called with |first_message|
   // for the result of this method.