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

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

Issue 983023002: [Chromoting] Use compact notation for javascript @private types (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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
« no previous file with comments | « remoting/webapp/crd/js/options_menu.js ('k') | remoting/webapp/crd/js/stats_accumulator.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/crd/js/session_connector_impl.js
diff --git a/remoting/webapp/crd/js/session_connector_impl.js b/remoting/webapp/crd/js/session_connector_impl.js
index fdbd6f0ccc6b1d7bf30f0ba261f301c90c2d4ac0..da0e9a822b3a75c5ab96dd61087447c7f33225b7 100644
--- a/remoting/webapp/crd/js/session_connector_impl.js
+++ b/remoting/webapp/crd/js/session_connector_impl.js
@@ -34,75 +34,40 @@ remoting.SessionConnectorImpl = function(clientContainer, onConnected, onError,
onConnectionFailed,
requiredCapabilities,
defaultRemapKeys) {
- /**
- * @type {HTMLElement}
- * @private
- */
+ /** @private {HTMLElement} */
this.clientContainer_ = clientContainer;
- /**
- * @type {function(remoting.ClientSession):void}
- * @private
- */
+ /** @private {function(remoting.ClientSession):void} */
this.onConnected_ = onConnected;
- /**
- * @type {function(remoting.Error):void}
- * @private
- */
+ /** @private {function(remoting.Error):void} */
this.onError_ = onError;
- /**
- * @type {function(string, string):boolean}
- * @private
- */
+ /** @private {function(string, string):boolean} */
this.onExtensionMessage_ = onExtensionMessage;
- /**
- * @type {function(remoting.Error):void}
- * @private
- */
+ /** @private {function(remoting.Error):void} */
this.onConnectionFailed_ = onConnectionFailed;
- /**
- * @type {Array<string>}
- * @private
- */
+ /** @private {Array<string>} */
this.requiredCapabilities_ = requiredCapabilities;
- /**
- * @type {string}
- * @private
- */
+ /** @private {string} */
this.defaultRemapKeys_ = defaultRemapKeys;
- /**
- * @type {string}
- * @private
- */
+ /** @private {string} */
this.clientJid_ = '';
- /**
- * @type {remoting.DesktopConnectedView.Mode}
- * @private
- */
+ /** @private {remoting.DesktopConnectedView.Mode} */
this.connectionMode_ = remoting.DesktopConnectedView.Mode.ME2ME;
- /**
- * @type {remoting.SignalStrategy}
- * @private
- */
+ /** @private {remoting.SignalStrategy} */
this.signalStrategy_ = null;
- /**
- * @type {remoting.SmartReconnector}
- * @private
- */
+ /** @private {remoting.SmartReconnector} */
this.reconnector_ = null;
- /**
- * @private
- */
+ /** @private */
this.bound_ = {
onStateChange : this.onStateChange_.bind(this)
};
@@ -116,34 +81,19 @@ remoting.SessionConnectorImpl = function(clientContainer, onConnected, onError,
* second connection. Note the none of the shared WCS state is reset.
*/
remoting.SessionConnectorImpl.prototype.reset = function() {
- /**
- * @type {remoting.Host}
- * @private
- */
+ /** @private {remoting.Host} */
this.host_ = null;
- /**
- * @type {boolean}
- * @private
- */
+ /** @private {boolean} */
this.logHostOfflineErrors_ = false;
- /**
- * @type {remoting.ClientSession}
- * @private
- */
+ /** @private {remoting.ClientSession} */
this.clientSession_ = null;
- /**
- * @type {XMLHttpRequest}
- * @private
- */
+ /** @private {XMLHttpRequest} */
this.pendingXhr_ = null;
- /**
- * @type {remoting.CredentialsProvider}
- * @private
- */
+ /** @type {remoting.CredentialsProvider} */
this.credentialsProvider_ = null;
};
« no previous file with comments | « remoting/webapp/crd/js/options_menu.js ('k') | remoting/webapp/crd/js/stats_accumulator.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698