Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(436)

Unified Diff: remoting/webapp/me2mom/client_session.js

Issue 8427020: Don't send a session-terminate in disconnect() if WCS not initialised. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/me2mom/client_session.js
diff --git a/remoting/webapp/me2mom/client_session.js b/remoting/webapp/me2mom/client_session.js
index 9ec8b4c8edceac207a8c39f8f99e545cb8a04667..1e8bfe1181dc6959cef266e4fbf2b8091d774f82 100644
--- a/remoting/webapp/me2mom/client_session.js
+++ b/remoting/webapp/me2mom/client_session.js
@@ -199,22 +199,22 @@ remoting.ClientSession.prototype.removePlugin = function() {
remoting.ClientSession.prototype.disconnect = function() {
if (remoting.wcs) {
remoting.wcs.setOnIq(function(stanza) {});
+ this.sendIq_(
+ '<cli:iq ' +
+ 'to="' + this.hostJid + '" ' +
+ 'type="set" ' +
+ 'id="session-terminate" ' +
+ 'xmlns:cli="jabber:client">' +
+ '<jingle ' +
+ 'xmlns="urn:xmpp:jingle:1" ' +
+ 'action="session-terminate" ' +
+ 'initiator="' + this.clientJid + '" ' +
+ 'sid="' + this.sessionId + '">' +
+ '<reason><success/></reason>' +
+ '</jingle>' +
+ '</cli:iq>');
}
this.removePlugin();
- this.sendIq_(
- '<cli:iq ' +
- 'to="' + this.hostJid + '" ' +
- 'type="set" ' +
- 'id="session-terminate" ' +
- 'xmlns:cli="jabber:client">' +
- '<jingle ' +
- 'xmlns="urn:xmpp:jingle:1" ' +
- 'action="session-terminate" ' +
- 'initiator="' + this.clientJid + '" ' +
- 'sid="' + this.sessionId + '">' +
- '<reason><success/></reason>' +
- '</jingle>' +
- '</cli:iq>');
};
/**
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698