[Chromoting] Request the daemon to open a terminal once a connection has been accepted.

remoting/host/client_session.h

Index: remoting/host/client_session.h
diff --git a/remoting/host/client_session.h b/remoting/host/client_session.h
index f0ab673e656cd9a06da760fb5b496a6a5fb31c22..6f939205519ae831279aa66fdb1356e2cdf85e4d 100644
--- a/remoting/host/client_session.h
+++ b/remoting/host/client_session.h
@@ -35,6 +35,7 @@ class AudioEncoder;
 class AudioScheduler;
 struct ClientSessionTraits;
 class DesktopEnvironment;
+class DesktopEnvironmentFactory;
 class ScreenRecorder;
 class VideoEncoder;
 class VideoFrameCapturer;
@@ -90,7 +91,7 @@ class ClientSession
                 scoped_refptr<base::SingleThreadTaskRunner> encode_task_runner,
                 scoped_refptr<base::SingleThreadTaskRunner> network_task_runner,
                 scoped_ptr<protocol::ConnectionToClient> connection,
-                scoped_ptr<DesktopEnvironment> desktop_environment,
+                DesktopEnvironmentFactory* desktop_environment_factory,

[Wez, 2012/10/17 22:04:24]

nit: Add a comment to clarify that |desktop_environment_factory_| is used only
in this ctor; otherwise it's not clear that passing an un-owned object to a
ref-counted one is safe!

On a related note, we should make sure our DesktopEnvironment[Factory]
implementations cope properly with the instances outliving the factory.

[alexeypa (please no reviews), 2012/10/17 22:46:56]

Done.

                 const base::TimeDelta& max_duration);
 
   // protocol::HostStub interface.