Block event processing on host/client until the client has authenticated.

remoting/client/chromoting_client.cc

 // Copyright (c) 2010 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.
 
 #include "remoting/client/chromoting_client.h"
 
 #include "base/message_loop.h"
 #include "remoting/base/tracer.h"
 #include "remoting/client/chromoting_view.h"
 #include "remoting/client/client_context.h"
@@ skipping 215 matching lines @@
 void ChromotingClient::BeginSessionResponse(
     const protocol::LocalLoginStatus* msg, Task* done) {
   if (message_loop() != MessageLoop::current()) {
     message_loop()->PostTask(
         FROM_HERE,
         NewRunnableMethod(this, &ChromotingClient::BeginSessionResponse,
                           msg, done));
     return;
   }
 
+  // Inform the connection that the client has been authenticated. This will
+  // enable the communication channels.
+  connection_->SetClientAuthenticated(msg->success());

[Wez, 2011/03/04 12:09:38]

I'd recommend SetClientAuthenticated(bool) -> if (bool) OnClientAuthenticated().
 Whether the client is authenticated should only ever go false->true.

[garykac, 2011/03/04 20:28:02]

Done.

+
   view_->UpdateLoginStatus(msg->success(), msg->error_info());
   done->Run();
   delete done;
 }
 
 }  // namespace remoting