[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 77 matching lines @@
  * Restart the window with |id|.
  * @param {string} id
  *
  * @private
  */
 remoting.ActivationHandler.prototype.onRestart_ = function(id) {
   this.appLauncher_.restart(id).then(this.onWindowCreated_.bind(this));
 };
 
 /**
+ * @param {Event} launchData
  * @private
  */
-remoting.ActivationHandler.prototype.onLaunched_ = function() {
-  this.createWindow_();
+remoting.ActivationHandler.prototype.onLaunched_ = function(launchData) {
+  if (launchData['isPublicSession']) {
+    this.launchPublicSession_();

[Jamie, 2015/09/30 20:24:09]

We're not really launching a public session; we're launching an auto-shared
IT2Me session. I think the name of this method should reflect what it does, not
the circumstances under which it's called.

Similarly, you might want to consider renaming createWindow_, since it's no
longer very descriptive.

Perhaps launchFullUi and launchIt2MeUi?

[kelvinp, 2015/09/30 23:13:49]

Done.

+  } else {
+    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));
 };
@@ skipping 25 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);
 };
 
+/** @private */
+remoting.ActivationHandler.prototype.launchPublicSession_ = function() {
+  chrome.app.window.create("public_session.html", {
+    'width': 570,
+    'height': 300,
+    'resizable': false
+  });
+};
+
 })();
 
 /** @enum {string} */
 remoting.ActivationHandler.Events = {
   windowClosed: 'windowClosed'
 };