Remove AccessVerifier interface.

remoting/host/client_session.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 REMOTING_HOST_CLIENT_SESSION_H_
 #define REMOTING_HOST_CLIENT_SESSION_H_
 
 #include <list>
 #include <set>
 
@@ skipping 13 matching lines @@
 class ClientSession : public protocol::HostStub,
                       public protocol::InputStub,
                       public protocol::ConnectionToClient::EventHandler,
                       public base::NonThreadSafe {
  public:
   // Callback interface for passing events to the ChromotingHost.
   class EventHandler {
    public:
     virtual ~EventHandler() {}
 
+    // Called after authentication has finished successfully.
     virtual void OnSessionAuthenticated(ClientSession* client) = 0;
+
+    // Called after authentication has failed. Must not tear down this
+    // object. Separate OnSessionClosed() is made after this handler

[Wez, 2011/11/29 05:58:47]

nit: Separate OnSessionClosed() is made ... -> OnSessionClosed is notified ...

[Sergey Ulanov, 2011/11/29 18:52:01]

Done.

+    // returns.
+    virtual void OnSessionAuthenticationFailed(ClientSession* client) = 0;
+
+    // Called if connection has failed or after the client closed it.

[Wez, 2011/11/29 05:58:47]

nit: if -> after?

[Sergey Ulanov, 2011/11/29 18:52:01]

Done.

     virtual void OnSessionClosed(ClientSession* client) = 0;
+
+    // Called for to notify of each message's sequence number. The

[Wez, 2011/11/29 05:58:47]

nit: for to notify of each message's ... -> to notify on receipt of a message

[Sergey Ulanov, 2011/11/29 18:52:01]

Done.

+    // callback cannot tear down this object.

[Wez, 2011/11/29 05:58:47]

nit: cannot -> must not

[Sergey Ulanov, 2011/11/29 18:52:01]

Done.

     virtual void OnSessionSequenceNumber(ClientSession* client,
                                          int64 sequence_number) = 0;
   };
 
   // Takes ownership of |connection|. Does not take ownership of
   // |event_handler|, |input_stub| or |capturer|.
   ClientSession(EventHandler* event_handler,
                 protocol::ConnectionToClient* connection,
                 protocol::InputStub* input_stub,
                 Capturer* capturer);
   virtual ~ClientSession();
 
   // protocol::InputStub interface.
   virtual void InjectKeyEvent(const protocol::KeyEvent& event) OVERRIDE;
   virtual void InjectMouseEvent(const protocol::MouseEvent& event) OVERRIDE;
 
   // protocol::ConnectionToClient::EventHandler interface.
   virtual void OnConnectionOpened(
       protocol::ConnectionToClient* connection) OVERRIDE;
   virtual void OnConnectionClosed(
       protocol::ConnectionToClient* connection) OVERRIDE;
-  virtual void OnConnectionFailed(
-      protocol::ConnectionToClient* connection) OVERRIDE;
+  virtual void OnConnectionFailed(protocol::ConnectionToClient* connection,
+                                  protocol::Session::Error error) OVERRIDE;
   virtual void OnSequenceNumberUpdated(
       protocol::ConnectionToClient* connection, int64 sequence_number) OVERRIDE;
 
   // Disconnects the session and destroys the transport. Event handler
   // is guaranteed not to be called after this method is called. Can
   // be called multiple times. The object should not be used after
   // this method returns.
   void Disconnect();
 
   protocol::ConnectionToClient* connection() const {
@@ skipping 70 matching lines @@
   // Set of keys that are currently pressed down by the user. This is used so
   // we can release them if the user disconnects.
   std::set<int> pressed_keys_;
 
   DISALLOW_COPY_AND_ASSIGN(ClientSession);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_CLIENT_SESSION_H_