[Chromoting] Create core Application interface for CRD.

remoting/webapp/base/js/application.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.
+
+/**
+ * @fileoverview
+ * Interface abstracting the Application functionality.
+ */
+
+'use strict';
+
+/** @suppress {duplicate} */
+var remoting = remoting || {};
+
+/**
+ * @constructor
+ *
+ * This class implements ApplicationDelegate so that the interface is
+ * consistent with the ApplicationDelegate's.
+ * @implements {remoting.ApplicationDelegate}

[Jamie, 2014/12/06 00:09:23]

Unless you're ever going to want to pass an instance of this class to something
that expects a Delegate, I would remove this; it feels circular. There's no harm
in using the same names without explicitly implementing the interface.

[garykac, 2014/12/06 01:39:33]

Done.

+ */
+remoting.Application = function() {
+  /**
+   * @type {remoting.ApplicationDelegate}
+   * @private
+   */
+  this.delegate_ = null;
+
+  /**
+   * @type {remoting.SessionConnector}
+   * @private
+   */
+  this.session_connector_ = null;
+};
+
+/**
+ * @param {remoting.ApplicationDelegate} appDelegate The delegate that
+ *    contains the app-specific functionality.
+ */
+remoting.Application.prototype.setDelegate = function(appDelegate) {
+  this.delegate_ = appDelegate;
+};
+
+/**
+ * Initialize the application and register all event handlers. After this
+ * is called, the app is running and waiting for user events.
+ *
+ * @return {void} Nothing.
+ */
+remoting.Application.prototype.init = function() {
+  this.delegate_.init();
+};
+
+/**
+ * Called when a new session has been connected.
+ *
+ * @param {remoting.ClientSession} clientSession
+ * @return {void} Nothing.
+ */
+remoting.Application.prototype.onConnected = function(clientSession) {
+  remoting.clientSession = clientSession;

[Jamie, 2014/12/06 00:09:23]

Long term, this should also be a member variable, but I suspect that would be a
lot of untangling for this CL. Add a TODO?

[garykac, 2014/12/06 01:39:33]

Added TODO.

+  remoting.clientSession.addEventListener('stateChanged', onClientStateChange_);
+
+  remoting.clipboard.startSession();
+  updateStatistics_();
+  remoting.hangoutSessionEvents.raiseEvent(
+      remoting.hangoutSessionEvents.sessionStateChanged,
+      remoting.ClientSession.State.CONNECTED
+  );
+
+  this.delegate_.onConnected(clientSession);
+};
+
+/**
+ * Called when the current session has been disconnected.
+ *
+ * @return {void} Nothing.
+ */
+remoting.Application.prototype.onDisconnected = function() {
+  this.delegate_.onDisconnected();
+};
+
+/**
+ * Called when the current session has reached the point where the host has
+ * started streaming video frames to the client.
+ *
+ * @return {void} Nothing.
+ */
+remoting.Application.prototype.onVideoStreamingStarted = function() {
+  this.delegate_.onVideoStreamingStarted();
+};
+
+/**
+ * Called when an extension message needs to be handled.
+ *
+ * @param {string} type The type of the extension message.
+ * @param {string} data The payload of the extension message.
+ * @return {boolean} Return true if the extension message was recognized.
+ */
+remoting.Application.prototype.onExtensionMessage = function(type, data) {
+  return this.delegate_.onExtensionMessage(type, data);
+};
+
+/**
+ * Called when an error needs to be displayed to the user.
+ *
+ * @param {remoting.Error} errorTag The error to be localized and displayed.
+ * @return {void} Nothing.
+ */
+remoting.Application.prototype.onError = function(errorTag) {
+  this.delegate_.onError(errorTag);
+};
+
+/**
+ * @return {remoting.SessionConnector} A session connector, creating a new one
+ *     if necessary.
+ */
+remoting.Application.prototype.getSessionConnector = function() {
+  if (!this.session_connector_) {
+    this.session_connector_ = remoting.SessionConnector.factory.createConnector(
+        document.getElementById('video-container'), this);
+  }
+  return this.session_connector_;
+};
+
+/** @type {remoting.Application} */
+remoting.app = null;