[Chromoting] Remote Assistance in Public Session.

remoting/webapp/crd/js/activation_handler.js

Index: remoting/webapp/crd/js/activation_handler.js
diff --git a/remoting/webapp/crd/js/activation_handler.js b/remoting/webapp/crd/js/activation_handler.js
index 901dde9f7562545b064557fbd1b249763b31bf8e..0b79bc1c1a85c5981672738dcdefea9d782f8841 100644
--- a/remoting/webapp/crd/js/activation_handler.js
+++ b/remoting/webapp/crd/js/activation_handler.js
@@ -95,10 +95,15 @@ remoting.ActivationHandler.prototype.onRestart_ = function(id) {
 };
 
 /**
+ * @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_();
+  }
 };
 
 /**
@@ -144,6 +149,15 @@ remoting.ActivationHandler.prototype.onWindowClosed_ = function(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} */