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

Unified Diff: remoting/webapp/crd/js/host_list.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/host_installer.js ('k') | remoting/webapp/crd/js/identity.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/crd/js/host_list.js
diff --git a/remoting/webapp/crd/js/host_list.js b/remoting/webapp/crd/js/host_list.js
index 5096998bd3f29e018fcf7a7ba044f3d259e7a1e8..a76cdc57653b98313e2e86f96293c230db8687bf 100644
--- a/remoting/webapp/crd/js/host_list.js
+++ b/remoting/webapp/crd/js/host_list.js
@@ -29,61 +29,33 @@ var remoting = remoting || {};
*/
remoting.HostList = function(table, noHosts, errorMsg, errorButton,
loadingIndicator) {
- /**
- * @type {Element}
- * @private
- */
+ /** @private {Element} */
this.table_ = table;
/**
- * @type {Element}
- * @private
* TODO(jamiewalch): This should be doable using CSS's sibling selector,
* but it doesn't work right now (crbug.com/135050).
+ * @private {Element}
*/
this.noHosts_ = noHosts;
- /**
- * @type {Element}
- * @private
- */
+ /** @private {Element} */
this.errorMsg_ = errorMsg;
- /**
- * @type {Element}
- * @private
- */
+ /** @private {Element} */
this.errorButton_ = errorButton;
- /**
- * @type {HTMLElement}
- * @private
- */
+ /** @private {HTMLElement} */
this.loadingIndicator_ = loadingIndicator;
- /**
- * @type {Array<remoting.HostTableEntry>}
- * @private
- */
+ /** @private {Array<remoting.HostTableEntry>} */
this.hostTableEntries_ = [];
- /**
- * @type {Array<remoting.Host>}
- * @private
- */
+ /** @private {Array<remoting.Host>} */
this.hosts_ = [];
- /**
- * @type {string}
- * @private
- */
+ /** @private {string} */
this.lastError_ = '';
- /**
- * @type {remoting.LocalHostSection}
- * @private
- */
+ /** @private {remoting.LocalHostSection} */
this.localHostSection_ = new remoting.LocalHostSection(
/** @type {HTMLElement} */ (document.querySelector('.daemon-control')),
new remoting.LocalHostSection.Controller(
this, new remoting.HostSetupDialog(remoting.hostController)));
- /**
- * @type {number}
- * @private
- */
+ /** @private {number} */
this.webappMajorVersion_ = parseInt(chrome.runtime.getManifest().version, 10);
this.errorButton_.addEventListener('click',
« no previous file with comments | « remoting/webapp/crd/js/host_installer.js ('k') | remoting/webapp/crd/js/identity.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698