Adding ability to stream windows and inject events to them

remoting/host/input_injector.h

Index: remoting/host/input_injector.h
diff --git a/remoting/host/input_injector.h b/remoting/host/input_injector.h
index ad0000a03a94b82ddbd7f4bcdf4e9e45b212f187..c7b04cce08bc9614cc7b0f6afad6dc39a212d1d3 100644
--- a/remoting/host/input_injector.h
+++ b/remoting/host/input_injector.h
@@ -9,7 +9,7 @@
 #include "base/memory/scoped_ptr.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;
 }  // namespace base
@@ -27,6 +27,12 @@ class InputInjector : public protocol::ClipboardStub,
       scoped_refptr<base::SingleThreadTaskRunner> main_task_runner,
       scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner);
 
+  static scoped_ptr<InputInjector> Create(

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

Name this "CreateForWindow". Chromium style doesn't allow overloading.

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

Done.

+      scoped_refptr<base::SingleThreadTaskRunner> main_task_runner,
+      scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner,
+      webrtc::WindowId windowId,
+      bool enable_window_capture);

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

This boolean flag is redundant. The calling code should call this method or the
other method to specify what kind of InputInjector to create.

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

Done.

+
   // Initialises any objects needed to execute events.
   virtual void Start(
       scoped_ptr<protocol::ClipboardStub> client_clipboard) = 0;