Use XMPP in V2 webapp

Use XMPP in V2 webapp

After this change the webapp will be use XMPP for signaling instead of
WCS.

BUG=274652
Committed: https://crrev.com/30975b8951c8c131883b0da0944b130fc8d1f2ae
Cr-Commit-Position: refs/heads/master@{#293273}

[Sergey Ulanov, 2014-09-03 00:14:30]

Patchset #1 (id:1) has been deleted

[Sergey Ulanov, 2014-09-03 00:18:22]

Patchset #2 (id:40001) has been deleted

[Sergey Ulanov, 2014-09-03 00:18:27]

Patchset #1 (id:20001) has been deleted

[Sergey Ulanov, 2014-09-03 00:18:57]

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

[Sergey Ulanov, 2014-09-03 00:42:42]

Patchset #2 (id:80001) has been deleted

[Wez, 2014-09-03 01:05:32]

wez@chromium.org changed reviewers:
+ wez@chromium.org

[Wez, 2014-09-03 01:05:33]

Does this patch support re-establishing a dropped XMPP session without changing
the JID?

[Sergey Ulanov, 2014-09-03 01:25:55]

On 2014/09/03 01:05:33, Wez wrote:
> Does this patch support re-establishing a dropped XMPP session without
changing
> the JID?

No, but it should be possible to implement it later. In this CL SessionConnecter
creates a new XMPP session for each client session.

[kelvinp, 2014-09-03 18:53:30]

lgtm with style nits.

https://codereview.chromium.org/530213004/diff/100001/remoting/webapp/client_...
File remoting/webapp/client_session.js (right):

https://codereview.chromium.org/530213004/diff/100001/remoting/webapp/client_...
remoting/webapp/client_session.js:872: /** @param{Element} message */
Nit space between @param and {Element}
Add private

https://codereview.chromium.org/530213004/diff/100001/remoting/webapp/session...
File remoting/webapp/session_connector.js (right):

https://codereview.chromium.org/530213004/diff/100001/remoting/webapp/session...
remoting/webapp/session_connector.js:346:
remoting.SessionConnector.prototype.connectSignaling_ = function() {
@private and same for other private functions.

[Sergey Ulanov, 2014-09-03 23:25:12]

https://codereview.chromium.org/530213004/diff/100001/remoting/webapp/client_...
File remoting/webapp/client_session.js (right):

https://codereview.chromium.org/530213004/diff/100001/remoting/webapp/client_...
remoting/webapp/client_session.js:872: /** @param{Element} message */
On 2014/09/03 18:53:29, kelvinp wrote:
> Nit space between @param and {Element}
> Add private

Done.

https://codereview.chromium.org/530213004/diff/100001/remoting/webapp/session...
File remoting/webapp/session_connector.js (right):

https://codereview.chromium.org/530213004/diff/100001/remoting/webapp/session...
remoting/webapp/session_connector.js:346:
remoting.SessionConnector.prototype.connectSignaling_ = function() {
On 2014/09/03 18:53:30, kelvinp wrote:
> @private and same for other private functions.

Done.

[Sergey Ulanov, 2014-09-03 23:26:07]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-09-03 23:28:36]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergeyu@chromium.org/530213004/120001

[commit-bot: I haz the power, 2014-09-04 02:30:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-04 02:30:33]

Try jobs failed on following builders:
  android_clang_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)

[Sergey Ulanov, 2014-09-04 04:22:33]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-09-04 04:23:52]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergeyu@chromium.org/530213004/120001

[commit-bot: I haz the power, 2014-09-04 05:44:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-04 05:44:52]

Try jobs failed on following builders:
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Sergey Ulanov, 2014-09-04 05:49:13]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-09-04 05:50:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergeyu@chromium.org/530213004/120001

[commit-bot: I haz the power, 2014-09-04 06:38:21]

Committed patchset #3 (id:120001) as 6f55b20cc5216bf8d14364ea4706eba0c0cbc8fb

[commit-bot: I haz the power, 2014-09-10 03:30:23]

Patchset 3 (id:??) landed as
https://crrev.com/30975b8951c8c131883b0da0944b130fc8d1f2ae
Cr-Commit-Position: refs/heads/master@{#293273}