Adding ability to stream windows and inject events to them

remoting/host/window_capturer_screen_wrapper.cc

+// Copyright (c) 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"
+
+namespace remoting {
+
+WindowCapturerScreenWrapper::WindowCapturerScreenWrapper() {

[Wez, 2014/08/01 23:41:55]

You're not initializing the |window_capturer_| bare pointer!

[ronakvora do not use, 2014/08/05 19:54:50]

Changed to scoped_ptr.

+}
+
+WindowCapturerScreenWrapper::~WindowCapturerScreenWrapper() {
+}
+
+bool WindowCapturerScreenWrapper::SelectWindow
+    (webrtc::WindowCapturer::WindowId id) {
+  return window_capturer_->SelectWindow(id);
+}
+void WindowCapturerScreenWrapper::SetWindow
+    (const webrtc::DesktopCaptureOptions& options) {
+  window_capturer_ = webrtc::WindowCapturer::Create(options);
+}
+
+void WindowCapturerScreenWrapper::Start(Callback* callback) {
+  window_capturer_->Start(callback);

[Wez, 2014/08/01 23:41:55]

When you switch |window_capturer_| to scoped_ptr<>, not only will it be
auto-initialized to NULL, it'll also DCHECK if you try to dereference it when
it's NULL.

[ronakvora do not use, 2014/08/05 19:54:50]

Got it.

+}
+
+void WindowCapturerScreenWrapper::Capture(const webrtc::DesktopRegion& region) {
+  window_capturer_->Capture(region);
+}
+
+void WindowCapturerScreenWrapper::SetMouseShapeObserver(
+      MouseShapeObserver* mouse_shape_observer) {

[Wez, 2014/08/01 23:41:55]

NOTIMPLEMENTED();

[ronakvora do not use, 2014/08/05 19:54:50]

This method gets called since we're using it as a ScreenCapturer. So, you can't
have it call NOTIMPLEMENTED(), can you?

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

[Wez, 2014/08/01 23:41:55]

NOTIMPLENTED();

[ronakvora do not use, 2014/08/05 19:54:50]

Even though it returns a boolean?

+  return false;
+}
+
+bool WindowCapturerScreenWrapper::SelectScreen(webrtc::ScreenId id) {

[Wez, 2014/08/01 23:41:55]

NOTIMPLEMENTED();

[ronakvora do not use, 2014/08/05 19:54:50]

Even though it returns a boolean?

+  return false;
+}
+
+}  // namespace remoting