[Chromoting] The desktop process now creates a pre-connected pipe and passes (with some help of the…

remoting/host/daemon_process.h

Index: remoting/host/daemon_process.h
diff --git a/remoting/host/daemon_process.h b/remoting/host/daemon_process.h
index c31d20fc897c910249ebcafba619a53608a87b64..1abd45afc3dc5dee01fdaf70f5cccd08c8353a08 100644
--- a/remoting/host/daemon_process.h
+++ b/remoting/host/daemon_process.h
@@ -12,8 +12,10 @@
 #include "base/location.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/process.h"
 #include "ipc/ipc_channel.h"
 #include "ipc/ipc_channel_proxy.h"
+#include "ipc/ipc_platform_file.h"
 #include "remoting/base/stoppable.h"
 #include "remoting/host/config_file_watcher.h"
 #include "remoting/host/worker_process_ipc_delegate.h"
@@ -62,6 +64,15 @@ class DaemonProcess
   // unless the network process is connected over the IPC channel.
   virtual void SendToNetwork(IPC::Message* message) = 0;
 
+  // Called when a desktop integration process attaches to |terminal_id|.
+  // |desktop_process| is a handle of the desktop integration process.
+  // |desktop_pipe| specifies the client end of the desktop pipe. Returns true
+  // on success, false otherwise.
+  virtual bool OnDesktopSessionAgentAttached(

[Sergey Ulanov, 2012/10/24 20:12:01]

Does this method need to be public? Maybe make it protected since we don't
expect it to be called from outside.

[alexeypa (please no reviews), 2012/10/24 21:41:51]

It is called by DesktopSessionWin::OnDesktopSessionAgentAttached().

+      int terminal_id,
+      base::ProcessHandle desktop_process,
+      IPC::PlatformFileForTransit desktop_pipe) = 0;
+
   // Closes the desktop session identified by |terminal_id|.
   void CloseDesktopSession(int terminal_id);