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

Side by Side 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 /** @suppress {duplicate} */
6 var remoting = remoting || {};
7
8 (function() {
9
10 'use strict';
11
12 /**
13 * An Activity is a responsible for
14 * 1. Showing the appropriate UX to establish a connection with the host and
15 * create a remoting.ClientSession.
16 * 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.
17 * 3. Responding to session state changes and showing UX if necessary.
18 *
19 * @interface
20 * @extends {base.Disposable}
21 * @extends {remoting.ClientSession.EventHandler}
22 */
23 remoting.Activity = function() {}
Jamie 2015/04/07 21:20:44 Nit: Terminating semicolon.
kelvinp 2015/04/07 22:48:19 Done.
24
25 /**
26 * Starts a new connection.
27 *
28 * @return {void}
29 */
30 remoting.Activity.prototype.start = function() {};
31
32 })();
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698