Fix XMPP late response.

Fix XMPP late response.

BUG=

Review-Url: https://codereview.chromium.org/2838993002
Cr-Commit-Position: refs/heads/master@{#468380}
Committed: https://chromium.googlesource.com/chromium/src/+/38b76371cb7055257e59dc386e831ecb8c2a6306

[Jamie, 2017-04-26 17:25:39]

jamiewalch@chromium.org changed reviewers:
+ kelvinp@chromium.org

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

I considered updating this to use whichever strategy succeeded first rather than
abandoning XMPP if it's slow to connect. Practically speaking, I don't think it
makes much difference (it counts for <1% of our connections), so I went with the
simpler option since this code won't be around much longer anyway, but LMK if
you disagree.

https://codereview.chromium.org/2838993002/diff/20001/remoting/webapp/base/js...
File remoting/webapp/base/js/fallback_signal_strategy.js (right):

https://codereview.chromium.org/2838993002/diff/20001/remoting/webapp/base/js...
remoting/webapp/base/js/fallback_signal_strategy.js:42:
this.PRIMARY_CONNECT_TIMEOUT_MS_ = 25 * 1000;
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.

[Jamie, 2017-04-27 22:13:53]

Description was changed from

==========
Fix XMPP late response.

BUG=
==========

to

==========
Fix XMPP late response.

BUG=
==========

[Jamie, 2017-04-27 22:13:53]

jamiewalch@chromium.org changed reviewers:
+ garykac@chromium.org
- kelvinp@chromium.org

[garykac, 2017-05-01 18:20:39]

lgtm

lgtm

(after much discussion ^_^)

[Jamie, 2017-05-01 18:23:02]

The CQ bit was checked by jamiewalch@chromium.org

[commit-bot: I haz the power, 2017-05-01 18:23:49]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-01 19:10:14]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493662982885060,
"parent_rev": "a7b31e1ff636ee016885a18953745783c9825616", "commit_rev":
"38b76371cb7055257e59dc386e831ecb8c2a6306"}

[commit-bot: I haz the power, 2017-05-01 19:10:48]

Description was changed from

==========
Fix XMPP late response.

BUG=
==========

to

==========
Fix XMPP late response.

BUG=

Review-Url: https://codereview.chromium.org/2838993002
Cr-Commit-Position: refs/heads/master@{#468380}
Committed:
https://chromium.googlesource.com/chromium/src/+/38b76371cb7055257e59dc386e83...
==========

[commit-bot: I haz the power, 2017-05-01 19:10:53]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/38b76371cb7055257e59dc386e83...