Adding ability to stream windows and inject events to them

remoting/host/window_capturer_screen_wrapper.cc

+// Copyright 2014 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.
+
+#include "remoting/host/window_capturer_screen_wrapper.h"

[Lambros, 2014/08/05 22:47:25]

nit: Blank line after this, as it's logically distinct from the other headers.

[ronakvora do not use, 2014/08/06 20:56:20]

Done.

+#include "base/logging.h"
+#include "third_party/webrtc/modules/desktop_capture/window_capturer.h"
+
+namespace remoting {
+
+WindowCapturerScreenWrapper::WindowCapturerScreenWrapper(
+    scoped_ptr<webrtc::WindowCapturer> window_capturer)
+    : window_capturer_(window_capturer.Pass()) {
+}
+
+WindowCapturerScreenWrapper::~WindowCapturerScreenWrapper() {
+}
+
+void WindowCapturerScreenWrapper::Start(Callback* callback) {
+  window_capturer_->Start(callback);
+}
+
+void WindowCapturerScreenWrapper::Capture(const webrtc::DesktopRegion& region) {
+  window_capturer_->Capture(region);
+}
+
+void WindowCapturerScreenWrapper::SetMouseShapeObserver(
+      MouseShapeObserver* mouse_shape_observer) {
+  return;

[Lambros, 2014/08/05 22:47:25]

Should be NOT_IMPLEMENTED() ?
nit: remove 'return;'

[ronakvora do not use, 2014/08/06 20:56:20]

Right now, we call this method. Is it alright if NOTIMPLEMENTED() gets called in
the code? It logs an error, but I'm not sure if that's okay for our purposes
right now.

+}
+bool WindowCapturerScreenWrapper::GetScreenList(ScreenList* screens) {

[Lambros, 2014/08/05 22:47:25]

nit: Blank line between methods.

[ronakvora do not use, 2014/08/06 20:56:20]

Done.

+  NOTIMPLEMENTED();
+  return false;
+}
+
+bool WindowCapturerScreenWrapper::SelectScreen(webrtc::ScreenId id) {
+  NOTIMPLEMENTED();
+  return false;
+}
+
+}  // namespace remoting