Preparations for Cast Host Extension

remoting/host/client_session.h

 // 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.
 
 #ifndef REMOTING_HOST_CLIENT_SESSION_H_
 #define REMOTING_HOST_CLIENT_SESSION_H_
 
 #include <string>
 
 #include "base/memory/ref_counted.h"
@@ skipping 12 matching lines @@
 #include "remoting/protocol/clipboard_echo_filter.h"
 #include "remoting/protocol/clipboard_filter.h"
 #include "remoting/protocol/clipboard_stub.h"
 #include "remoting/protocol/connection_to_client.h"
 #include "remoting/protocol/host_stub.h"
 #include "remoting/protocol/input_event_tracker.h"
 #include "remoting/protocol/input_filter.h"
 #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;
 }  // namespace base
 
 namespace remoting {
 
 class AudioEncoder;
 class AudioScheduler;
 class DesktopEnvironment;
@@ skipping 115 matching lines @@
   protocol::ConnectionToClient* connection() const {
     return connection_.get();
   }
 
   bool is_authenticated() { return auth_input_filter_.enabled();  }
 
   const std::string* client_capabilities() const {
     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;
 
   // Creates a proxy for sending clipboard events to the client.
   scoped_ptr<protocol::ClipboardStub> CreateClipboardProxy();
 
   // Creates an audio encoder for the specified configuration.
   static scoped_ptr<AudioEncoder> CreateAudioEncoder(
       const protocol::SessionConfig& config);
 
@@ skipping 81 matching lines @@
 
   // The pairing registry for PIN-less authentication.
   scoped_refptr<protocol::PairingRegistry> pairing_registry_;
 
   // Used to proxy gnubby auth traffic.
   scoped_ptr<GnubbyAuthHandler> gnubby_auth_handler_;
 
   // 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);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_CLIENT_SESSION_H_