diff options
| author | sanjeevr@chromium.org <sanjeevr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-26 06:27:35 +0000 |
|---|---|---|
| committer | sanjeevr@chromium.org <sanjeevr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-26 06:27:35 +0000 |
| commit | 01bb55975be133c852242549d2099655f5e1cc08 (patch) | |
| tree | 0c3d331ffe4ca6f31352016a955846e2b3cd350e /jingle | |
| parent | 85d9d2b6b87ef093556aa3c92a262dbe4c1b73aa (diff) | |
| download | chromium_src-01bb55975be133c852242549d2099655f5e1cc08.zip chromium_src-01bb55975be133c852242549d2099655f5e1cc08.tar.gz chromium_src-01bb55975be133c852242549d2099655f5e1cc08.tar.bz2 | |
Removed the JID check from PushNotificationsListenTask::IsValidNotification. This was causing issues because the case did not match in some instances. We could do a case-insensitive check but we really do not need to check the TO: JID here.
BUG=None
TEST=Cloud Printing should work with users who signed up for Gmail specifying a non-lowercase username.
Review URL: http://codereview.chromium.org/6309017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72612 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'jingle')
| -rw-r--r-- | jingle/notifier/listener/push_notifications_listen_task.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/jingle/notifier/listener/push_notifications_listen_task.cc b/jingle/notifier/listener/push_notifications_listen_task.cc index ecf7b95..90b5c0a 100644 --- a/jingle/notifier/listener/push_notifications_listen_task.cc +++ b/jingle/notifier/listener/push_notifications_listen_task.cc @@ -92,8 +92,9 @@ bool PushNotificationsListenTask::IsValidNotification( // <data>{base-64 data}</data> // </push> // </message> - return ((stanza->Name() == buzz::QN_MESSAGE) && - (stanza->Attr(buzz::QN_TO) == GetClient()->jid().Str())); + // We don't do much validation here, just check if the stanza is a message + // stanza. + return (stanza->Name() == buzz::QN_MESSAGE); } } // namespace notifier |