[Sync] Relax checks for received XMPP messages

chrome/browser/sync/notifier/cache_invalidation_packet_handler.cc

Index: chrome/browser/sync/notifier/cache_invalidation_packet_handler.cc
diff --git a/chrome/browser/sync/notifier/cache_invalidation_packet_handler.cc b/chrome/browser/sync/notifier/cache_invalidation_packet_handler.cc
index be46ecadf6a045b9c388e4b594de0210e1b21a9e..49875a69145196c68937f8000cf76ee3fca45414 100644
--- a/chrome/browser/sync/notifier/cache_invalidation_packet_handler.cc
+++ b/chrome/browser/sync/notifier/cache_invalidation_packet_handler.cc
@@ -84,15 +84,7 @@ class CacheInvalidationListenTask : public buzz::XmppTask {
 
  private:
   bool IsValidCacheInvalidationIqPacket(const buzz::XmlElement* stanza) {
-    // We make sure to compare jids (which are normalized) instead of
-    // just strings -- server may use non-normalized jids in
-    // attributes.
-    //
-    // TODO(akalin): Add unit tests for this.
-    buzz::Jid to(stanza->Attr(buzz::QN_TO));
-    return
-        (MatchRequestIq(stanza, buzz::STR_SET, kQnData) &&
-         (to == GetClient()->jid()));
+    return MatchRequestIq(stanza, buzz::STR_SET, kQnData);

[sanjeevr, 2011/01/30 22:55:07]

Nit: Might want to add some comments mentioning the reasoning behind the limited
validation.

[akalin, 2011/01/31 09:54:17]

Done.

   }
 
   bool GetCacheInvalidationIqPacketData(const buzz::XmlElement* stanza,