Adding ability to stream windows and inject events to them

remoting/host/single_window_input_injector.h

+// 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.
+
+#ifndef REMOTING_HOST_SINGLE_WINDOW_INPUT_INJECTOR_H_
+#define REMOTING_HOST_SINGLE_WINDOW_INPUT_INJECTOR_H_
+
+#include "base/memory/ref_counted.h"

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

Many of these headers aren't needed.

Here, you just need:
* scoped_ptr.h for scoped_ptr.
* input_injector.h for base class definition.
* webrtc header for webrtc::WindowId typedef.

Any other headers you need for the .cc belong in the .cc.

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

Done.

+#include "base/memory/scoped_ptr.h"
+#include "remoting/host/input_injector.h"
+#include "remoting/protocol/clipboard_stub.h"
+#include "remoting/protocol/input_stub.h"
+#include "third_party/webrtc/modules/desktop_capture/desktop_capture_types.h"
+
+namespace base {
+class SingleThreadTaskRunner;

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

Remove this.

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

Done.

+}  // namespace base
+
+namespace remoting {
+
+class SingleWindowInputInjector : public InputInjector {

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

Please add a comment here explaining the purpose of this class.

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

Done.

+ public:
+  static scoped_ptr<InputInjector> Create(
+      webrtc::WindowId window_id,
+      scoped_ptr<InputInjector> input_injector);

[Wez, 2014/08/06 04:10:16]

nit: Either clarify that |input_injector| normally needs to be passed a
full-desktop InputInjector, or just create it internally in the
SingleWindowInputInjector. :)

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

Done.

+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_SINGLE_WINDOW_INPUT_INJECTOR_H_