Touch Events capability negotiation

remoting/host/basic_desktop_environment.cc

Index: remoting/host/basic_desktop_environment.cc
diff --git a/remoting/host/basic_desktop_environment.cc b/remoting/host/basic_desktop_environment.cc
index e7132f655782431684c9dd69086d5ef2ccd6a42a..be2e0778237b6464eb47c1df6bdcfcd81481c02f 100644
--- a/remoting/host/basic_desktop_environment.cc
+++ b/remoting/host/basic_desktop_environment.cc
@@ -22,6 +22,10 @@
 
 namespace remoting {
 
+namespace {
+const char kRawTouchEventsCapability[] = "rawTouchEvents";

[Wez, 2015/04/23 22:10:25]

Why "raw" touch events?

I think it'd be reasonable to call this touchEvents and we can then call a
subsequent addition touchEventsV2, touchGestures, touchActions, etc, as
necessary. :)

nit: Is there really no common location we can move this definition to? :-/

[Rintaro Kuroiwa, 2015/04/24 01:55:02]

Make sense :) will go with touchEvents.

I saw kRateLimitResizeRequests being duplicated so I also duplicated, maybe I'm
wrong because I only see kRateLimitResizeRequests in Me2MeDE and
DesktopSessionProxy (essentially IpcDE) which are both Me2Me. I guess the
translation unit scope prevents from accidentally setting it for It2Me sessions.

Seems overkill to add 'host_capabilities.h' just for this capability. Do you
think input_injector.h would be a reasonable place?

+}  // namespace
+
 BasicDesktopEnvironment::~BasicDesktopEnvironment() {
   DCHECK(caller_task_runner_->BelongsToCurrentThread());
 }
@@ -55,6 +59,9 @@ BasicDesktopEnvironment::CreateMouseCursorMonitor() {
 }
 
 std::string BasicDesktopEnvironment::GetCapabilities() const {
+  if (supports_raw_touch_events_)
+    return kRawTouchEventsCapability;
+
   return std::string();
 }
 
@@ -89,7 +96,8 @@ BasicDesktopEnvironment::BasicDesktopEnvironment(
       ui_task_runner_(ui_task_runner),
       desktop_capture_options_(
           new webrtc::DesktopCaptureOptions(
-              webrtc::DesktopCaptureOptions::CreateDefault())) {
+              webrtc::DesktopCaptureOptions::CreateDefault())),
+      supports_raw_touch_events_(false) {
   DCHECK(caller_task_runner_->BelongsToCurrentThread());
 }
 
@@ -99,7 +107,8 @@ BasicDesktopEnvironmentFactory::BasicDesktopEnvironmentFactory(
     scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner)
     : caller_task_runner_(caller_task_runner),
       input_task_runner_(input_task_runner),
-      ui_task_runner_(ui_task_runner) {
+      ui_task_runner_(ui_task_runner),
+      supports_raw_touch_events_(false) {
 }
 
 BasicDesktopEnvironmentFactory::~BasicDesktopEnvironmentFactory() {