[Webapp Refactor] Fix styles in SmartReconnector.

remoting/webapp/crd/js/smart_reconnector.js

 // Copyright 2014 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.
 
 /**
  * @fileoverview
  * Class handling reconnecting the session when it is disconnected due to
  * network failure.
  *
  * The SmartReconnector listens for changes in connection state of
  * |clientSession| to determine if a reconnection is needed.  It then calls into
  * |connector| to reconnect the session.
  */
 
-'use strict';
-
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
+(function () {
+
+'use strict';
+
 /**
  * @constructor
  * @param {function()} reconnectCallback
  * @param {function()} disconnectCallback
  * @param {remoting.ClientSession} clientSession This represents the current
  *    remote desktop connection.  It is used to monitor the changes in
  *    connection state.
  * @implements {base.Disposable}
  */
 remoting.SmartReconnector =
     function(reconnectCallback, disconnectCallback, clientSession) {
   /** @private */
   this.reconnectCallback_ = reconnectCallback;
 
   /** @private */
   this.disconnectCallback_ = disconnectCallback;
 
   /** @private */
   this.clientSession_ = clientSession;
 
+  /** @private {base.Disposable} */
+  this.pending_ = null;

[Jamie, 2015/04/16 20:05:49]

You're using this for two separate pending operations, which forces you to give
it a nondescript name. How do you feel about using separate members with better
names?

[kelvinp, 2015/04/16 20:57:58]

|this.pending_| is intended to be a placeholder of a pending reconnect
operation that is cancel-able, regardless of the type. 

+
+  var Events = remoting.ClientSession.Events;
   /** @private */
-  this.reconnectTimerId_ = null;
-
-  /** @private */
-  this.connectionTimeoutTimerId_ = null;
-
-  /** @private */
-  this.bound_ = {
-    reconnect: this.reconnect_.bind(this),
-    reconnectAsync: this.reconnectAsync_.bind(this),
-    startReconnectTimeout: this.startReconnectTimeout_.bind(this),
-    stateChanged: this.stateChanged_.bind(this),
-    videoChannelStateChanged: this.videoChannelStateChanged_.bind(this)
-  };
-
-  clientSession.addEventListener(
-      remoting.ClientSession.Events.stateChanged,
-      this.bound_.stateChanged);
-  clientSession.addEventListener(
-      remoting.ClientSession.Events.videoChannelStateChanged,
-      this.bound_.videoChannelStateChanged);
+  this.eventHooks_ = new base.Disposables(
+      new base.EventHook(clientSession, Events.stateChanged,
+                         this.stateChanged_.bind(this)),
+      new base.EventHook(clientSession, Events.videoChannelStateChanged,
+                         this.videoChannelStateChanged_.bind(this)));
 };
 
 // The online event only means the network adapter is enabled, but
 // it doesn't necessarily mean that we have a working internet connection.
-// Therefore, delay the connection by |kReconnectDelay| to allow for the network
-// to connect.
-remoting.SmartReconnector.kReconnectDelay = 2000;
+// Therefore, delay the connection by |RECONNECT_DELAY_MS| to allow for the
+// network to connect.
+var RECONNECT_DELAY_MS = 2000;
 
 // If the video channel is inactive for 10 seconds reconnect the session.
-remoting.SmartReconnector.kConnectionTimeout = 10000;
+var CONNECTION_TIMEOUT_MS = 10000;
 
-remoting.SmartReconnector.prototype = {
-  reconnect_: function() {
-    this.cancelPending_();
-    this.disconnectCallback_();
-    remoting.setMode(remoting.AppMode.CLIENT_CONNECTING);
-    this.reconnectCallback_();
-  },
+remoting.SmartReconnector.prototype.reconnect_ = function() {
+  base.dispose(this.pending_);

[Jamie, 2015/04/16 20:05:49]

Reset pending_ to null, here and below.

[kelvinp, 2015/04/16 20:57:58]

Done.

+  this.disconnectCallback_();
+  remoting.setMode(remoting.AppMode.CLIENT_CONNECTING);
+  this.reconnectCallback_();
+};
 
-  reconnectAsync_: function() {
-    this.cancelPending_();
-    remoting.setMode(remoting.AppMode.CLIENT_CONNECTING);
-    this.reconnectTimerId_ = window.setTimeout(
-        this.bound_.reconnect, remoting.SmartReconnector.kReconnectDelay);
-  },
+remoting.SmartReconnector.prototype.reconnectAsync_ = function() {
+  base.dispose(this.pending_);
+  remoting.setMode(remoting.AppMode.CLIENT_CONNECTING);
+  this.pending_ =
+      new base.OneShotTimer(this.reconnect_.bind(this), RECONNECT_DELAY_MS);
+};
 
-  /**
-   * @param {remoting.ClientSession.StateEvent=} event
-   */
-  stateChanged_: function(event) {
-    var State = remoting.ClientSession.State;
-    if (event.previous === State.CONNECTED && event.current === State.FAILED) {
-      this.cancelPending_();
-      if (navigator.onLine) {
-        this.reconnect_();
-      } else {
-        window.addEventListener('online', this.bound_.reconnectAsync, false);
-      }
+/**
+ * @param {remoting.ClientSession.StateEvent=} event
+ */
+remoting.SmartReconnector.prototype.stateChanged_ = function(event) {
+  var State = remoting.ClientSession.State;
+  if (event.previous === State.CONNECTED && event.current === State.FAILED) {
+    base.dispose(this.pending_);
+    if (navigator.onLine) {
+      this.reconnect_();
+    } else {
+      this.pending_ = new base.DomEventHook(
+          window, 'online', this.reconnectAsync_.bind(this), false);
     }
-  },
-
-  /**
-   * @param {boolean=} active  True if the video channel is active.
-   */
-  videoChannelStateChanged_: function (active) {
-    this.cancelPending_();
-    if (!active) {
-      window.addEventListener(
-          'online', this.bound_.startReconnectTimeout, false);
-    }
-  },
-
-  startReconnectTimeout_: function () {
-    this.cancelPending_();
-    this.connectionTimeoutTimerId_ = window.setTimeout(
-          this.bound_.reconnect, remoting.SmartReconnector.kConnectionTimeout);
-  },
-
-  cancelPending_: function() {
-    window.removeEventListener(
-        'online', this.bound_.startReconnectTimeout, false);
-    window.removeEventListener('online', this.bound_.reconnectAsync, false);
-    window.clearTimeout(this.reconnectTimerId_);
-    window.clearTimeout(this.connectionTimeoutTimerId_);
-    this.reconnectTimerId_ = null;
-    this.connectionTimeoutTimerId_ = null;
-  },
-
-  dispose: function() {
-    this.clientSession_.removeEventListener(
-        remoting.ClientSession.Events.stateChanged,
-        this.bound_.stateChanged);
-    this.clientSession_.removeEventListener(
-        remoting.ClientSession.Events.videoChannelStateChanged,
-        this.bound_.videoChannelStateChanged);
   }
 };
+
+/**
+ * @param {boolean=} active  True if the video channel is active.
+ */
+remoting.SmartReconnector.prototype.videoChannelStateChanged_ =
+    function (active) {
+  base.dispose(this.pending_);
+  if (!active) {
+      this.pending_ = new base.DomEventHook(

[Jamie, 2015/04/16 20:05:49]

Indentation.

[kelvinp, 2015/04/16 20:57:58]

Done.

[Jamie, 2015/04/17 00:02:47]

This still looks wrong. This line is indented by 4 spaces w.r.t. the previous,
not 2.

[kelvinp, 2015/04/17 00:12:53]

My bad.  My this.pending_ change overwrote the indenting fix.  Now fixed.

+          window, 'online', this.startReconnectTimeout_.bind(this), false);
+  }
+};
+
+remoting.SmartReconnector.prototype.startReconnectTimeout_ = function() {
+  base.dispose(this.pending_);
+  this.pending_ =
+      new base.OneShotTimer(this.reconnect_.bind(this), CONNECTION_TIMEOUT_MS);
+};
+
+remoting.SmartReconnector.prototype.dispose = function() {
+  base.dispose(this.pending_);
+  this.pending_ = null;
+  base.dispose(this.eventHooks_);
+  this.eventHooks_ = null;
+};
+
+})();