Added support of Secure Attention Sequence in multiprocess mode.

remoting/host/desktop_session_win.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.
 
 #include "remoting/host/desktop_session_win.h"
 
 #include "base/path_service.h"
 #include "ipc/ipc_message_macros.h"
 #include "remoting/base/auto_thread_task_runner.h"
 #include "remoting/host/chromoting_messages.h"
 #include "remoting/host/daemon_process.h"
+#include "remoting/host/sas_injector.h"
 #include "remoting/host/win/worker_process_launcher.h"
 #include "remoting/host/win/wts_console_monitor.h"
 #include "remoting/host/win/wts_session_process_delegate.h"
 
 using base::win::ScopedHandle;
 
 namespace {
 
 const FilePath::CharType kDesktopBinaryName[] =
     FILE_PATH_LITERAL("remoting_desktop.exe");
@@ skipping 43 matching lines @@
   VLOG(1) << "IPC: daemon <- desktop (" << peer_pid << ")";
 }
 
 bool DesktopSessionWin::OnMessageReceived(const IPC::Message& message) {
   DCHECK(main_task_runner_->BelongsToCurrentThread());
 
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(DesktopSessionWin, message)
     IPC_MESSAGE_HANDLER(ChromotingDesktopDaemonMsg_DesktopAttached,
                         OnDesktopSessionAgentAttached)
+    IPC_MESSAGE_HANDLER(ChromotingDesktopDaemonMsg_InjectSas,
+                        OnInjectSas)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
 void DesktopSessionWin::OnPermanentError() {
   DCHECK(main_task_runner_->BelongsToCurrentThread());
 
   launcher_.reset();
 
@@ skipping 36 matching lines @@
 
 void DesktopSessionWin::OnDesktopSessionAgentAttached(
       IPC::PlatformFileForTransit desktop_pipe) {
   if (!daemon_process()->OnDesktopSessionAgentAttached(id(),
                                                        desktop_process_,
                                                        desktop_pipe)) {
     RestartDesktopProcess(FROM_HERE);
   }
 }
 
+void DesktopSessionWin::OnInjectSas() {
+  DCHECK(main_task_runner_->BelongsToCurrentThread());
+
+  if (!launcher_)
+    return;

[Jamie, 2012/12/06 21:42:25]

This check seems unnecessary, since you're not using launcher in this function.

[alexeypa (please no reviews), 2012/12/06 21:56:02]

This check is required for the case when OnInjectSas arrived, but suddenly the
desktop process dies for any reason.

[Jamie, 2012/12/06 22:07:09]

Okay, maybe a comment would be useful to indicate what a NULL launcher_
indicates, and why it makes sense not to inject SAS in this case.

[alexeypa (please no reviews), 2012/12/06 22:38:51]

Done.

+
+  if (!sas_injector_)
+    sas_injector_ = SasInjector::Create();
+  if (!sas_injector_->InjectSas())
+    LOG(ERROR) << "Failed to inject Secure Attention Sequence.";
+}
+
 void DesktopSessionWin::RestartDesktopProcess(
     const tracked_objects::Location& location) {
   DCHECK(main_task_runner_->BelongsToCurrentThread());
 
   launcher_->Send(new ChromotingDaemonDesktopMsg_Crash(
       location.function_name(), location.file_name(), location.line_number()));
 }
 
 }  // namespace remoting