Fix XMPP late response.

remoting/webapp/base/js/fallback_signal_strategy.js

 // Copyright 2015 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.
 
 'use strict';
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 /**
@@ skipping 21 matching lines @@
   /** @private {?function(remoting.SignalStrategy.State)} */
   this.onStateChangedCallback_ = null;
 
   /** @private {?function(Element):void} */
   this.onIncomingStanzaCallback_ = null;
 
   /**
    * @private {number}
    * @const
    */
-  this.PRIMARY_CONNECT_TIMEOUT_MS_ = 10 * 1000;
+  this.PRIMARY_CONNECT_TIMEOUT_MS_ = 25 * 1000;

[Jamie, 2017/04/26 17:25:40]

Looking at the stats, there's a big spike in XMPP/SUCCEEDED_LATE logs around
20s. Since XMPP is a expected to be more reliable than WCS, delaying WCS to give
the XMPP connection longer to succeed seems like a good trade-off.

 
   /**
    * @enum {string}
    * @private
    */
   this.State = {
     NOT_CONNECTED: 'not-connected',
     PRIMARY_PENDING: 'primary-pending',
     PRIMARY_SUCCEEDED: 'primary-succeeded',
     SECONDARY_PENDING: 'secondary-pending',
@@ skipping 171 matching lines @@
       if (this.state_ == this.State.PRIMARY_PENDING) {
         window.clearTimeout(this.primaryConnectTimerId_);
         this.updateProgress_(
             this.primary_,
             remoting.FallbackSignalStrategy.Progress.SUCCEEDED);
         this.state_ = this.State.PRIMARY_SUCCEEDED;
       } else {
         this.updateProgress_(
             this.primary_,
             remoting.FallbackSignalStrategy.Progress.SUCCEEDED_LATE);
+        return;  // Don't notify the external callback
       }
       break;
 
     case remoting.SignalStrategy.State.FAILED:
       if (this.state_ == this.State.PRIMARY_PENDING) {
         window.clearTimeout(this.primaryConnectTimerId_);
         this.updateProgress_(
             this.primary_,
             remoting.FallbackSignalStrategy.Progress.FAILED);
         this.connectSecondary_();
@@ skipping 89 matching lines @@
  * @param {remoting.SignalStrategy} strategy
  * @param {remoting.FallbackSignalStrategy.Progress} progress
  * @private
  */
 remoting.FallbackSignalStrategy.prototype.updateProgress_ = function(
     strategy, progress) {
   console.log('FallbackSignalStrategy progress: ' + strategy.getType() + ' ' +
       progress);
   this.logger_.logSignalStrategyProgress(strategy.getType(), progress);
 };