Remove authenticated_ fields from stubs.

remoting/protocol/client_message_dispatcher.cc

Index: remoting/protocol/client_message_dispatcher.cc
diff --git a/remoting/protocol/client_message_dispatcher.cc b/remoting/protocol/client_message_dispatcher.cc
index a1020d2b00fe6018aac56c51ebdd14f570a0f867..e0c2758ac7b62b4218f37e84a47a72e98e20fdde 100644
--- a/remoting/protocol/client_message_dispatcher.cc
+++ b/remoting/protocol/client_message_dispatcher.cc
@@ -39,30 +39,19 @@ void ClientMessageDispatcher::Initialize(
 
 void ClientMessageDispatcher::OnControlMessageReceived(
     ControlMessage* message, Task* done_task) {
-  if (!client_stub_->authenticated()) {
-    // When the client has not authenticated with the host, we restrict the
-    // control messages that we support.
-    if (message->has_begin_session_response()) {
-      client_stub_->BeginSessionResponse(
-          &message->begin_session_response().login_status(), done_task);
-      return;
-    } else {
-      LOG(WARNING) << "Invalid control message received "
-                   << "(client not authenticated).";
-    }
+  // TODO(sergeyu): Add message validation.
+  if (message->has_notify_resolution()) {
+    client_stub_->NotifyResolution(
+        &message->notify_resolution(), done_task);
+    return;
+  } else if (message->has_begin_session_response()) {
+    client_stub_->BeginSessionResponse(
+        &message->begin_session_response().login_status(), done_task);
+    return;
   } else {
-    // TODO(sergeyu): Add message validation.
-    if (message->has_notify_resolution()) {
-      client_stub_->NotifyResolution(
-          &message->notify_resolution(), done_task);
-      return;
-    } else if (message->has_begin_session_response()) {
-      LOG(WARNING) << "BeginSessionResponse sent after client already "
-                   << "authorized.";
-    } else {
-      LOG(WARNING) << "Invalid control message received.";
-    }
+    LOG(WARNING) << "Invalid control message received.";

[Wez, 2011/03/30 20:24:57]

If not authenticated, disconnect?

[simonmorris, 2011/03/31 11:14:00]

Could be good, in another CL.

   }
+
   done_task->Run();
   delete done_task;
 }