Remove redundant comments

Author: mmermerkaya

packages/database/src/core/PersistentConnection.ts

@@ -76,7 +76,6 @@ export class PersistentConnection extends ServerActions {
   id = PersistentConnection.nextPersistentConnectionId_++;
   private log_ = logWrapper('p:' + this.id + ':');
 
-  /** @private {Object} */
   private interruptReasons_: { [reason: string]: boolean } = {};
   private listens_: { [path: string]: { [queryId: string]: ListenSpec } } = {};
   private outstandingPuts_: OutstandingPut[] = [];
@@ -88,26 +87,19 @@ export class PersistentConnection extends ServerActions {
   private securityDebugCallback_: ((a: Object) => void) | null = null;
   lastSessionId: string | null = null;
 
-  /** @private {number|null} */
   private establishConnectionTimer_: number | null = null;
 
-  /** @private {boolean} */
   private visible_: boolean = false;
 
   // Before we get connected, we keep a queue of pending messages to send.
   private requestCBHash_: { [k: number]: (a: any) => void } = {};
   private requestNumber_ = 0;
 
-  /** @private {?{
-   *   sendRequest(Object),
-   *   close()
-   * }} */
   private realtime_: {
     sendRequest(a: Object): void;
     close(): void;
   } | null = null;
 
-  /** @private {string|null} */
   private authToken_: string | null = null;
   private forceTokenRefresh_ = false;
   private invalidAuthTokenCount_ = 0;
@@ -116,26 +108,17 @@ export class PersistentConnection extends ServerActions {
   private lastConnectionAttemptTime_: number | null = null;
   private lastConnectionEstablishedTime_: number | null = null;
 
-  /**
-   * @private
-   */
   private static nextPersistentConnectionId_ = 0;
 
   /**
    * Counter for number of connections created. Mainly used for tagging in the logs
-   * @type {number}
-   * @private
    */
   private static nextConnectionId_ = 0;
 
   /**
    * @implements {ServerActions}
-   * @param {!RepoInfo} repoInfo_ Data about the namespace we are connecting to
-   * @param {function(string, *, boolean, ?number)} onDataUpdate_ A callback for new data from the server
-   * @param onConnectStatus_
-   * @param onServerInfoUpdate_
-   * @param authTokenProvider_
-   * @param authOverride_
+   * @param repoInfo_ Data about the namespace we are connecting to
+   * @param onDataUpdate_ A callback for new data from the server
    */
   constructor(
     private repoInfo_: RepoInfo,
@@ -166,12 +149,6 @@ export class PersistentConnection extends ServerActions {
     }
   }
 
-  /**
-   * @param {!string} action
-   * @param {*} body
-   * @param {function(*)=} onResponse
-   * @protected
-   */
   protected sendRequest(
     action: string,
     body: any,
@@ -226,13 +203,6 @@ export class PersistentConnection extends ServerActions {
     }
   }
 
-  /**
-   * @param {!{onComplete(),
-   *           hashFn():!string,
-   *           query: !Query,
-   *           tag: ?number}} listenSpec
-   * @private
-   */
   private sendListen_(listenSpec: ListenSpec) {
     const query = listenSpec.query;
     const pathString = query.path.toString();
@@ -274,11 +244,6 @@ export class PersistentConnection extends ServerActions {
     });
   }
 
-  /**
-   * @param {*} payload
-   * @param {!Query} query
-   * @private
-   */
   private static warnOnListenWarnings_(payload: any, query: Query) {
     if (payload && typeof payload === 'object' && contains(payload, 'w')) {
       const warnings = safeGet(payload, 'w');
@@ -319,10 +284,6 @@ export class PersistentConnection extends ServerActions {
     this.reduceReconnectDelayIfAdminCredential_(token);
   }
 
-  /**
-   * @param {!string} credential
-   * @private
-   */
   private reduceReconnectDelayIfAdminCredential_(credential: string) {
     // NOTE: This isn't intended to be bulletproof (a malicious developer can always just modify the client).
     // Additionally, we don't bother resetting the max delay back to the default if auth fails / expires.
@@ -576,10 +537,6 @@ export class PersistentConnection extends ServerActions {
     }
   }
 
-  /**
-   * @param {*} message
-   * @private
-   */
   private onDataMessage_(message: { [k: string]: any }) {
     if ('r' in message) {
       // this is a response
@@ -663,10 +620,6 @@ export class PersistentConnection extends ServerActions {
     }, Math.floor(timeout)) as any;
   }
 
-  /**
-   * @param {boolean} visible
-   * @private
-   */
   private onVisible_(visible: boolean) {
     // NOTE: Tabbing away and back to a window will defeat our reconnect backoff, but I think that's fine.
     if (
@@ -819,9 +772,6 @@ export class PersistentConnection extends ServerActions {
     }
   }
 
-  /**
-   * @param {string} reason
-   */
   interrupt(reason: string) {
     log('Interrupting connection for reason: ' + reason);
     this.interruptReasons_[reason] = true;
@@ -838,9 +788,6 @@ export class PersistentConnection extends ServerActions {
     }
   }
 
-  /**
-   * @param {string} reason
-   */
   resume(reason: string) {
     log('Resuming connection for reason: ' + reason);
     delete this.interruptReasons_[reason];
@@ -872,11 +819,6 @@ export class PersistentConnection extends ServerActions {
     if (this.outstandingPutCount_ === 0) this.outstandingPuts_ = [];
   }
 
-  /**
-   * @param {!string} pathString
-   * @param {Array.<*>=} query
-   * @private
-   */
   private onListenRevoked_(pathString: string, query?: any[]) {
     // Remove the listen and manufacture a "permission_denied" error for the failed listen.
     let queryId;
@@ -889,12 +831,6 @@ export class PersistentConnection extends ServerActions {
     if (listen && listen.onComplete) listen.onComplete('permission_denied');
   }
 
-  /**
-   * @param {!string} pathString
-   * @param {!string} queryId
-   * @return {{queries:Array.<Query>, onComplete:function(string)}}
-   * @private
-   */
   private removeListen_(pathString: string, queryId: string): ListenSpec {
     const normalizedPathString = new Path(pathString).toString(); // normalize path.
     let listen;
@@ -971,7 +907,6 @@ export class PersistentConnection extends ServerActions {
 
   /**
    * Sends client stats for first connection
-   * @private
    */
   private sendConnectStats_() {
     const stats: { [k: string]: number } = {};
@@ -995,10 +930,6 @@ export class PersistentConnection extends ServerActions {
     this.reportStats(stats);
   }
 
-  /**
-   * @return {boolean}
-   * @private
-   */
   private shouldReconnect_(): boolean {
     const online = OnlineMonitor.getInstance().currentlyOnline();
     return isEmpty(this.interruptReasons_) && online;