[Webapp Refactor] Implements remoting.Activity.

remoting/webapp/crd/js/connect_flow.js

+// Copyright 2015 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';
+
+/**
+ * @interface
+ * @extends {base.Disposable}
+ */
+remoting.ConnectFlow = function() {}
+
+/**
+ * Starts a new connection.
+ *
+ * @return {void}
+ */
+remoting.ConnectFlow.prototype.start = function() {};
+
+/**
+ * Called when the connection failed before it is connected.
+ *
+ * @param {!remoting.Error} error
+ */
+remoting.ConnectFlow.prototype.onConnectionFailed = function(error) {};

[Jamie, 2015/04/03 19:54:23]

This interface is a bit confusing. I expected to see a start() method, perhaps
returning a Promise to indicate success/failure. The start() method clearly
belongs on the interface, because it's how a client of this API uses it, but the
other methods seem more like callbacks--they're how an implementation of this
interface notifies its client of various events.

+
+/**
+ * Called when a new session has been connected.
+ *
+ * @param {!remoting.ConnectionInfo} connectionInfo
+ */
+remoting.ConnectFlow.prototype.onConnected = function(connectionInfo) {};
+
+/**
+ * Called when the current session has been disconnected.
+ */
+remoting.ConnectFlow.prototype.onDisconnected = function() {};
+
+/**
+ * Called when an error needs to be displayed to the user.
+ * @param {!remoting.Error} error
+ */
+remoting.ConnectFlow.prototype.onError = function(error) {};
+
+})();