Preparations for Cast Host Extension

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 a86839ec5875e133f5a6f87a987829cd112aeb40..6e71bfb0aaf351ac757ba83876ca7250f6a7d979 100644
--- a/remoting/host/client_session.h
+++ b/remoting/host/client_session.h
@@ -30,6 +30,7 @@
 #include "remoting/protocol/input_stub.h"
 #include "remoting/protocol/pairing_registry.h"
 #include "third_party/webrtc/modules/desktop_capture/desktop_geometry.h"
+#include "third_party/webrtc/modules/desktop_capture/screen_capturer.h"
 
 namespace base {
 class SingleThreadTaskRunner;
@@ -165,6 +166,18 @@ class ClientSession
     return client_capabilities_.get();
   }
 
+  void set_enable_cast(bool enable) { enable_cast_ = enable; }

[Wez, 2014/07/17 22:56:37]

As for ChromotingHost, clarify what this actually means.

+
+  scoped_refptr<base::SingleThreadTaskRunner> network_task_runner() {
+    return network_task_runner_;
+  }
+  scoped_refptr<base::SingleThreadTaskRunner> video_capture_task_runner() {
+    return video_capture_task_runner_;
+  }
+  scoped_ptr<webrtc::ScreenCapturer> RequestScreenCapturer() {

[Wez, 2014/07/17 22:56:36]

This actually releases and returns the ScreenCapturer - why do we need that
here?

+    return screen_capturer_.Pass();
+  }
+
  private:
   typedef ScopedVector<HostExtensionSession> HostExtensionSessionList;
 
@@ -266,6 +279,13 @@ class ClientSession
   // Host extension sessions, used to handle extension messages.
   HostExtensionSessionList extension_sessions_;
 
+  // True if cast mode is enabled.
+  bool enable_cast_;
+
+  // Used to capture screens. This is a class member to allow external objects,

[Wez, 2014/07/17 22:56:37]

nit: "Used to capture screens" isn't very helpful ;) You might say "Used to
capture desktop contents" or something along those lines.

+  // like CastExtensionSession, to request it when needed.

[Wez, 2014/07/17 22:56:37]

Use by what? Does the normal Chromoting video channel use it, for example?

+  scoped_ptr<webrtc::ScreenCapturer> screen_capturer_;
+
   DISALLOW_COPY_AND_ASSIGN(ClientSession);
 };