[Chromoting] Remote Assistance in Public Session.

remoting/webapp/crd/js/activation_handler.js

 // Copyright 2014 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.
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 (function(){
 
 'use strict';
@@ skipping 89 matching lines @@
 remoting.ActivationHandler.prototype.onLaunched_ = function() {
   this.createWindow_();
 };
 
 /**
  * Create a new app window and register for the closed event.
  *
  * @private
  */
 remoting.ActivationHandler.prototype.createWindow_ = function() {
-  this.appLauncher_.launch().then(this.onWindowCreated_.bind(this));
+  var that = this;
+  return this.isPublicSession_().then(
+    this.launchPublicSession_.bind(this)
+  ).catch(function() {
+    that.appLauncher_.launch().then(that.onWindowCreated_.bind(that));
+  });
 };
 
 /**
  * @param {string} windowId
  *
  * @private
  */
 remoting.ActivationHandler.prototype.onWindowCreated_ = function(
     windowId) {
   // Send the client heartbeat.
@@ skipping 16 matching lines @@
  */
 remoting.ActivationHandler.prototype.onWindowClosed_ = function(id) {
   // Unhook the event.
   var hook = /** @type {base.Disposable} */ (this.windowClosedHooks_.get(id));
   base.dispose(hook);
   this.windowClosedHooks_.delete(id);
 
   this.raiseEvent(remoting.ActivationHandler.Events.windowClosed, id);
 };
 
+/**
+ * @return {Promise} A Promise that resolves if this is a public session or
+ *     rejects if it is not.
+ * @private
+ */
+remoting.ActivationHandler.prototype.isPublicSession_ = function() {
+  // Check whether this is a service account to determine whether we are
+  // currently signed-in as a public session or not.
+  return remoting.identity.getEmail().then(function(/** string */ email) {
+    if (email.indexOf('@gserviceaccount.com') == -1) {

[Jamie, 2015/09/30 17:46:13]

Please add a TODO to clean this up once we have an API (you may be able to
de-Promisify it at that point).

[kelvinp, 2015/09/30 19:35:47]

Fixed to use the actual flag instead.

+      return Promise.reject();
+    }
+  });
+};
+
+/** @private */
+remoting.ActivationHandler.prototype.launchPublicSession_ = function() {
+  chrome.app.window.create("public_session.html", {
+    'width': 570,
+    'height': 300,

[Jamie, 2015/09/30 17:46:13]

In general, you can't hard-code a height because it will be too small for some
languages and too large for others. I think you need to call resizeToContent
instead, probably each time the DOM changes.

[kelvinp, 2015/09/30 19:35:47]

Fixed in public_session_main.js

+    'resizable': false
+  });
+};
+
 })();
 
 /** @enum {string} */
 remoting.ActivationHandler.Events = {
   windowClosed: 'windowClosed'
 };