Touch Events capability negotiation

remoting/webapp/crd/js/client_session.js

 // Copyright (c) 2012 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.
 
 /**
  * @fileoverview
  * Class handling creation and teardown of a remoting client session.
  *
  * The ClientSession class controls lifetime of the client plugin
  * object and provides the plugin with the functionality it needs to
@@ skipping 189 matching lines @@
   // resolution to the host once connection has been established. See
   // this.plugin_.notifyClientResolution().
   SEND_INITIAL_RESOLUTION: 'sendInitialResolution',
 
   // Let the host know that we're interested in knowing whether or not it
   // rate limits desktop-resize requests.
   // TODO(kelvinp): This has been supported since M-29.  Currently we only have
   // <1000 users on M-29 or below. Remove this and the capability on the host.
   RATE_LIMIT_RESIZE_REQUESTS: 'rateLimitResizeRequests',
 
+  // Host & client support touch. Client can always support multi touch
+  // because if the client is not a touch device, it cannot send touch events to
+  // the host. What really matters is whether the host supports this.
+  // If not the plugin should not handle touch events and get synthesized
+  // mouse events from PPAPI.

[Wez, 2015/03/30 17:10:07]

Suggest rewording:

"Host & client support touch. If the host does not support touch then the client
will let PPAPI translate touch input into mouse events, for compatibility with
non-touch-aware UI."

[Rintaro Kuroiwa, 2015/04/08 03:27:43]

Done.

+  MULTI_TOUCH: 'multiTouch',
+
   // Indicates that host/client supports Google Drive integration, and that the
   // client should send to the host the OAuth tokens to be used by Google Drive
   // on the host.
   GOOGLE_DRIVE: "googleDrive",
 
   // Indicates that the client supports the video frame-recording extension.
   VIDEO_RECORDER: 'videoRecorder',
 
   // Indicates that the client supports 'cast'ing the video stream to a
   // cast-enabled device.
@@ skipping 238 matching lines @@
 remoting.ClientSession.prototype.onSetCapabilities = function(capabilities) {
   if (this.capabilities_ != null) {
     console.error('onSetCapabilities_() is called more than once');
     return;
   }
 
   this.capabilities_ = capabilities;
   if (this.hasCapability(remoting.ClientSession.Capability.GOOGLE_DRIVE)) {
     this.sendGoogleDriveAccessToken_();
   }
+

[Wez, 2015/03/30 17:10:07]

nit: No need for blank line.

[Rintaro Kuroiwa, 2015/04/08 03:27:43]

Done.

+  if (this.hasCapability(remoting.ClientSession.Capability.MULTI_TOUCH)) {
+    this.plugin_.enableTouchEvents();
+  }
 };
 
 /**
  * @param {string} type
  * @param {string} data
  */
 remoting.ClientSession.prototype.onExtensionMessage = function(type, data) {
   this.onExtensionMessageHandler_(type, data);
 };
 
@@ skipping 117 matching lines @@
   var googleDriveScopes = [
     'https://docs.google.com/feeds/',
     'https://www.googleapis.com/auth/drive'
   ];
   remoting.identity.getNewToken(googleDriveScopes).
       then(sendToken).
       catch(remoting.Error.handler(sendError));
   window.setTimeout(this.sendGoogleDriveAccessToken_.bind(this),
                     remoting.ACCESS_TOKEN_RESEND_INTERVAL_MS);
 };
-