Adding ability to stream windows and inject events to them

remoting/host/window_capturer_screen_wrapper.h

+#include "third_party/webrtc/modules/desktop_capture/screen_capturer.h"

[Lambros, 2014/07/30 00:14:50]

Missing copyright notice.

[ronakvora do not use, 2014/07/30 20:55:37]

Done.

+#include "third_party/webrtc/modules/desktop_capture/window_capturer.h"
+
+namespace remoting {
+
+class WindowCapturerScreenWrapper : public ::webrtc::ScreenCapturer {
+ public:
+  explicit WindowCapturerScreenWrapper();

[Lambros, 2014/07/30 00:14:50]

"explicit" not needed.

[ronakvora do not use, 2014/07/30 20:55:37]

Done.

+  virtual  ~WindowCapturerScreenWrapper();
+
+  /*
+  // WindowCapturer interface.
+  virtual bool GetWindowList(WindowList* windows);
+  virtual bool BringSelectedWindowToFront();
+  */
+  // WindowCapturer interface.

[Lambros, 2014/07/30 00:14:50]

This is confusing, because this class doesn't actually derive in any way from
WindowCapturer. I don't think these methods should be declared virtual.

[ronakvora do not use, 2014/07/30 20:55:37]

Done.

+  virtual bool SelectWindow(webrtc::WindowId id);
+  virtual void SetWindow(const webrtc::DesktopCaptureOptions& options);
+
+  // DesktopCapturer interface.
+  virtual void Start(DesktopCapturer::Callback* callback) OVERRIDE;
+  virtual void Capture(const webrtc::DesktopRegion& region) OVERRIDE;
+
+  // webrtc::ScreenCapturer interface.
+  virtual void SetMouseShapeObserver(
+      MouseShapeObserver* mouse_shape_observer) OVERRIDE;
+  virtual bool GetScreenList(ScreenList* screens) OVERRIDE;
+  virtual bool SelectScreen(webrtc::ScreenId id) OVERRIDE;
+  static WindowCapturerScreenWrapper* Create();

[Lambros, 2014/07/30 00:14:50]

This method isn't needed. Remove it, and update the call-site to call "new
WindowCapturerScreenWrapper()" directly.

[ronakvora do not use, 2014/07/30 20:55:37]

Done.

+
+ private:
+  webrtc::WindowCapturer* window_capturer_;
+
+  DISALLOW_COPY_AND_ASSIGN(WindowCapturerScreenWrapper);
+};
+
+
+}  // namespace remoting