Use the local user's email when connecting to It2MeHelpeeChannel

remoting/webapp/crd/js/identity.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
  * Wrapper class for Chrome's identity API.
  */
 
 'use strict';
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 /**
  * TODO(jamiewalch): Remove remoting.OAuth2 from this type annotation when
  * the Apps v2 work is complete.
  *
  * @type {remoting.Identity}
  */
 remoting.identity = null;
 
 /**
- * @param {remoting.Identity.ConsentDialog} consentDialog
+ * @param {remoting.Identity.ConsentDialog=} consentDialog

[dcaiafa, 2015/02/04 00:12:35]

nit: According to the Google style guide, optional javascript parameters should
start with the "opt_" prefix. I don't know how closely we follow the guide,
though.

[kelvinp, 2015/02/04 00:38:13]

Done.

  * @constructor
  */
 remoting.Identity = function(consentDialog) {
   /** @private */
   this.consentDialog_ = consentDialog;
   /** @type {string} @private */
   this.email_ = '';
   /** @type {string} @private */
   this.fullName_ = '';
   /** @type {Array.<remoting.Identity.Callbacks>} */
   this.pendingCallbacks_ = [];
 };
 
 /**
- * chrome.identity.getAuthToken must be initiated from user interactions if
+ * chrome.identity.getAuthToken should be initiated from user interactions if
  * called with interactive equals true.  This interface prompts a dialog for
  * the user's consent.
  *
  * @interface
  */
 remoting.Identity.ConsentDialog = function() {};
 
 /**
  * @return {Promise} A Promise that resolves when permission to start an
  *   interactive flow is granted.
@@ skipping 166 matching lines @@
       var callback = /** @type {remoting.Identity.Callbacks} */
           (this.pendingCallbacks_.shift());
       callback.onError(remoting.Error.NOT_AUTHENTICATED);
     }
     return;
   }
 
   // If there's no token, but we haven't yet prompted for permission, do so
   // now.
   var that = this;
-  this.consentDialog_.show().then(function() {
+  var showConsentDialog =
+      (this.consentDialog_) ? this.consentDialog_.show() : Promise.resolve();
+  showConsentDialog.then(function() {
     chrome.identity.getAuthToken({'interactive': true},
                                  that.onAuthComplete_.bind(that, true));
   });
 };
 
 /**
  * Internal representation for pair of callWithToken callbacks.
  *
  * @param {function(string):void} onOk
  * @param {function(remoting.Error):void} onError
  * @constructor
  * @private
  */
 remoting.Identity.Callbacks = function(onOk, onError) {
   /** @type {function(string):void} */
   this.onOk = onOk;
   /** @type {function(remoting.Error):void} */
   this.onError = onError;
 };
 
 /**
  * Returns whether the web app has authenticated with the Google services.
  *
  * @return {boolean}
  */
 remoting.Identity.prototype.isAuthenticated = function() {
   return remoting.identity.email_ !== '';
 };