The authenticated_ fields are moved out of stubs and into

ClientSession. Messages to the stubs are dispatched via
ClientSession, and the stub classes are pure virtual.

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/6724033

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79991 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 23 deletions

diff --git a/remoting/protocol/host_stub.h b/remoting/protocol/host_stub.h
index 737154d..2a18ffe 100644
--- a/remoting/protocol/host_stub.h
+++ b/remoting/protocol/host_stub.h
@@ -4,7 +4,7 @@
 
 // Interface of a host that receives commands from a Chromoting client.
 //
-// This interface handles control messages defined in contro.proto.
+// This interface handles control messages defined in control.proto.
 
 #ifndef REMOTING_PROTOCOL_HOST_STUB_H_
 #define REMOTING_PROTOCOL_HOST_STUB_H_
@@ -21,35 +21,15 @@ class SuggestResolutionRequest;
 
 class HostStub {
  public:
-  HostStub();
-  virtual ~HostStub();
+  HostStub() {};
+  virtual ~HostStub() {};
 
   virtual void SuggestResolution(
       const SuggestResolutionRequest* msg, Task* done) = 0;
   virtual void BeginSessionRequest(
       const LocalLoginCredentials* credentials, Task* done) = 0;
 
-  // TODO(lambroslambrou): Remove OnAuthenticated() and OnClosed() when stubs
-  // are refactored not to store authentication state.
-
-  // Called when the client has authenticated with the host to enable the
-  // client->host control channel.
-  // Before this is called, only a limited set of control messages will be
-  // processed.
-  void OnAuthenticated();
-
-  // Called when the client is no longer connected.
-  void OnClosed();
-
-  // Has the client successfully authenticated with the host?
-  // I.e., should we be processing control events?
-  bool authenticated();
-
  private:
-  // Initially false, this records whether the client has authenticated with
-  // the host.
-  bool authenticated_;
-
   DISALLOW_COPY_AND_ASSIGN(HostStub);
 };