Hangout remote desktop part I - It2Me mode

remoting/webapp/hangout_session.js

Index: remoting/webapp/hangout_session.js
diff --git a/remoting/webapp/hangout_session.js b/remoting/webapp/hangout_session.js
new file mode 100644
index 0000000000000000000000000000000000000000..05f2bc4c89ad852b7296aafc7a59f28d22f41ae8
--- /dev/null
+++ b/remoting/webapp/hangout_session.js
@@ -0,0 +1,73 @@
+// 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.
+
+/**
+ * @fileoverview
+ * Class to communicate with the background scripts via chrome runtime
+ * messages to
+ * 1. Forward session state notifications
+ * 2. Closes the window when the session terminates
+ */
+
+'use strict';
+
+/** @suppress {duplicate} */
+var remoting = remoting || {};
+
+/**
+ * @constructor
+ */
+remoting.HangoutSession = function() {
+  /**
+   * @private
+   * @type {chrome.extension.Port}
+   */
+  this.port_ = null;
+};
+
+remoting.HangoutSession.prototype.init = function() {
+  this.port_ = chrome.runtime.connect({name: 'it2me.helper.webapp'});
+
+  remoting.hangoutSessionEvents.addEventListener(
+      remoting.hangoutSessionEvents.sessionStateChanged,
+      this.onSessionStateChanged_.bind(this));
+};
+
+/**
+ * @param {remoting.ClientSession.State} state
+ */
+remoting.HangoutSession.prototype.onSessionStateChanged_ = function(state) {
+  var State = remoting.ClientSession.State;
+  try {
+    this.port_.postMessage({method: 'sessionStateChanged', state: state});
+  } catch (e) {
+    // postMessage will throw an exception if the port is disconnected.
+    // We can safely ignore this exception.
+  } finally {
+    if (state === State.FAILED || state === State.CLOSED) {
+      // close the current window
+      if (remoting.isAppsV2) {
+        chrome.app.window.current().close();
+      } else {
+        window.close();
+      }
+    }
+  }
+};
+
+
+

[Jamie, 2014/08/08 01:24:12]

Three blank lines is overkill :) I suggest two, since there's a substantial
context switch taking place.

[kelvinp, 2014/08/08 18:58:50]

Done.

+/**
+ * remoting.clientSession does not exist until the session is connected.
+ * hangoutSessionEvents serves as a global event source to plumb session
+ * state changes until we cleanup clientSession and sessionConnector.
+ * @type {base.EventSource}
+ */
+remoting.hangoutSessionEvents = new base.EventSource();
+
+/** @type {string} */
+remoting.hangoutSessionEvents.sessionStateChanged = "sessionStateChanged";
+
+remoting.hangoutSessionEvents.defineEvents(
+    [remoting.hangoutSessionEvents.sessionStateChanged]);