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

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

Issue 8568031: Call plugin.onIq() only when onIq is not null. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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 9fcd4d950ff4663a3e80086f841eee7fc4b2c1ff..9de2f37fa0df6ea19e8a93d581b8c53d6f410970 100644
--- a/remoting/webapp/me2mom/client_session.js
+++ b/remoting/webapp/me2mom/client_session.js
@@ -272,7 +272,8 @@ remoting.ClientSession.prototype.connectPluginToWcs_ =
/** @param {string} stanza The IQ stanza received. */
var onIq = function(stanza) {
remoting.debug.logIq(false, stanza);
- that.plugin.onIq(stanza);
+ if (that.plugin.onIq)
Wez 2011/11/19 00:59:14 Do we want to log something different if there is
garykac 2011/11/19 01:12:49 +1 We shouldn't be logging the stanza if we didn't
Sergey Ulanov 2011/11/19 01:30:53 This function handles incoming messages, not outgo
Sergey Ulanov 2011/11/19 01:30:53 We don't send anything here, it's for incoming mes
+ that.plugin.onIq(stanza);
}
remoting.wcs.setOnIq(onIq);
that.plugin.connect(this.hostJid, this.hostPublicKey, this.clientJid,
« 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