Commit 01bb5597 authored by sanjeevr@chromium.org's avatar sanjeevr@chromium.org

Removed the JID check from PushNotificationsListenTask::IsValidNotification....

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
parent 85d9d2b6
......@@ -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
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment