Touch Events capability negotiation

remoting/host/basic_desktop_environment.h

Index: remoting/host/basic_desktop_environment.h
diff --git a/remoting/host/basic_desktop_environment.h b/remoting/host/basic_desktop_environment.h
index 6b6b8e25ede0e94a14c63af789926a45a7a8e130..e7fc0b4aa798ae0def0bb733c9844ebf510e7165 100644
--- a/remoting/host/basic_desktop_environment.h
+++ b/remoting/host/basic_desktop_environment.h
@@ -40,6 +40,9 @@ class BasicDesktopEnvironment : public DesktopEnvironment {
   scoped_ptr<GnubbyAuthHandler> CreateGnubbyAuthHandler(
       protocol::ClientStub* client_stub) override;
 
+  // Enable or disables multi-touch capability.
+  void SetEnableMultiTouch(bool enable);

[Wez, 2015/04/21 02:12:12]

Why is this called multi-touch here and touch-events elsewhere?

Since this is a simple setter it should be in-lined and named
set_enable_multi_touch().

Does the setter really need to be public?

[Rintaro Kuroiwa, 2015/04/22 19:55:59]

Since I'm changing the capabilities string to rawTouchEvent (see other comment),
I'll make this set_supports_raw_touch_event().
It still doesn't quite align with InjectTouchEvent(). Maybe I should rename it
:/

+
  protected:
   friend class BasicDesktopEnvironmentFactory;
 
@@ -64,6 +67,8 @@ class BasicDesktopEnvironment : public DesktopEnvironment {
     return desktop_capture_options_.get();
   }
 
+  bool multi_touch_enabled() const { return multi_touch_enabled_; }
+
  private:
   // Task runner on which methods of DesktopEnvironment interface should be
   // called.
@@ -82,6 +87,8 @@ class BasicDesktopEnvironment : public DesktopEnvironment {
   // find build errors.
   scoped_ptr<webrtc::DesktopCaptureOptions> desktop_capture_options_;
 
+  bool multi_touch_enabled_;
+
   DISALLOW_COPY_AND_ASSIGN(BasicDesktopEnvironment);
 };
 
@@ -96,6 +103,7 @@ class BasicDesktopEnvironmentFactory : public DesktopEnvironmentFactory {
 
   // DesktopEnvironmentFactory implementation.
   bool SupportsAudioCapture() const override;
+  void SetEnableMultiTouch(bool enable) override;
 
  protected:
   scoped_refptr<base::SingleThreadTaskRunner> caller_task_runner() const {
@@ -110,6 +118,8 @@ class BasicDesktopEnvironmentFactory : public DesktopEnvironmentFactory {
     return ui_task_runner_;
   }
 
+  bool multi_touch_enabled() const { return multi_touch_enabled_; }
+
  private:
   // Task runner on which methods of DesktopEnvironmentFactory interface should
   // be called.
@@ -121,6 +131,8 @@ class BasicDesktopEnvironmentFactory : public DesktopEnvironmentFactory {
   // Used to run UI code.
   scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner_;
 
+  bool multi_touch_enabled_;
+
   DISALLOW_COPY_AND_ASSIGN(BasicDesktopEnvironmentFactory);
 };