Use RejectAuthenticatingClient instead of DisconnectAllClients to reject incoming clients on curtai…

remoting/host/remoting_me2me_host.cc

 // Copyright (c) 2012 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.
 //
 // This file implements a standalone host process for Me2Me.
 
 #include <string>
 
 #include "base/at_exit.h"
 #include "base/bind.h"
@@ skipping 118 matching lines @@
 #else  // !defined(OS_WIN)
         desktop_environment_factory_(new DesktopEnvironmentFactory(
             context_->input_task_runner(), context_->ui_task_runner())),
 #endif  // !defined(OS_WIN)
         desktop_resizer_(DesktopResizer::Create()),
         exit_code_(kSuccessExitCode) {
     network_change_notifier_.reset(net::NetworkChangeNotifier::Create());
     curtain_ = CurtainMode::Create(
         base::Bind(&HostProcess::OnDisconnectRequested,
                    base::Unretained(this)),
-        base::Bind(&HostProcess::OnDisconnectRequested,
+        base::Bind(&HostProcess::RejectAuthenticatingClient,
                    base::Unretained(this)));
 }
 
 #if defined(REMOTING_MULTI_PROCESS)
 
   bool InitWithCommandLine(const CommandLine* cmd_line) {
     // Parse the handle value and convert it to a handle/file descriptor.
     std::string channel_name =
         cmd_line->GetSwitchValueASCII(kDaemonPipeSwitchName);
 
@@ skipping 445 matching lines @@
 
     host_->Start(xmpp_login_);
 
     CreateAuthenticatorFactory();
   }
 
   void OnAuthFailed() {
     Shutdown(kInvalidOauthCredentialsExitCode);
   }
 
+  void RejectAuthenticatingClient() {
+    DCHECK(context_->network_task_runner()->BelongsToCurrentThread());
+    DCHECK(host_);

[rmsousa, 2012/10/17 01:03:55]

Specifically, this must always be called from an OnClientAuthenticated callback
(this specific DCHECK is done inside host_->RejectAuthenticatingClient())

+    host_->RejectAuthenticatingClient();
+  }
+
   // Invoked when the user uses the Disconnect windows to terminate
   // the sessions, or when the local session is activated in curtain mode.
   void OnDisconnectRequested() {
     if (!context_->network_task_runner()->BelongsToCurrentThread()) {
       context_->network_task_runner()->PostTask(FROM_HERE, base::Bind(
           &HostProcess::OnDisconnectRequested, base::Unretained(this)));
       return;
     }
     if (host_) {
       host_->DisconnectAllClients();
@@ skipping 214 matching lines @@
             user32.GetFunctionPointer("SetProcessDPIAware"));
     set_process_dpi_aware();
   }
 
   // CommandLine::Init() ignores the passed |argc| and |argv| on Windows getting
   // the command line from GetCommandLineW(), so we can safely pass NULL here.
   return main(0, NULL);
 }
 
 #endif  // defined(OS_WIN)