Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(228)

Unified Diff: remoting/webapp/crd/js/activity.js

Issue 1054273002: [Webapp Refactor] Implements remoting.Activity. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reviewer's feedback Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/webapp/crd/js/activity.js
diff --git a/remoting/webapp/crd/js/activity.js b/remoting/webapp/crd/js/activity.js
new file mode 100644
index 0000000000000000000000000000000000000000..b07d2a17eab1b09678e76d50155f9cfd500374a2
--- /dev/null
+++ b/remoting/webapp/crd/js/activity.js
@@ -0,0 +1,32 @@
+// 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';
+
+/**
+ * An Activity is a responsible for
+ * 1. Showing the appropriate UX to establish a connection with the host and
+ * create a remoting.ClientSession.
+ * 2. Handling connection failure and retry if necessary.
Jamie 2015/04/07 21:20:44 s/failure/failures/ s/retry/retrying/
kelvinp 2015/04/07 22:48:19 Done.
+ * 3. Responding to session state changes and showing UX if necessary.
+ *
+ * @interface
+ * @extends {base.Disposable}
+ * @extends {remoting.ClientSession.EventHandler}
+ */
+remoting.Activity = function() {}
Jamie 2015/04/07 21:20:44 Nit: Terminating semicolon.
kelvinp 2015/04/07 22:48:19 Done.
+
+/**
+ * Starts a new connection.
+ *
+ * @return {void}
+ */
+remoting.Activity.prototype.start = function() {};
+
+})();

Powered by Google App Engine
This is Rietveld 408576698