Added class to subscribe to GCD notifications over XMPP.

remoting/signaling/jid_util.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/signaling/jid_util.h"
 
 #include "base/strings/string_util.h"
 
 namespace remoting {
 
 std::string NormalizeJid(const std::string& jid) {
-  size_t slash_pos = jid.find('/');
+  std::string bare_jid;
+  std::string resource_suffix;
+  SplitJidResource(jid, &bare_jid, &resource_suffix);
+  return base::StringToLowerASCII(bare_jid) + resource_suffix;
+}
 
-  // In case there the jid doesn't have resource id covert the whole value to
-  // lower-case.
-  if (slash_pos == std::string::npos)
-    return base::StringToLowerASCII(jid);
-
-  return base::StringToLowerASCII(jid.substr(0, slash_pos)) +
-         jid.substr(slash_pos);
+bool SplitJidResource(const std::string& full_jid,
+                      std::string* bare_jid,
+                      std::string* resource_suffix) {
+  size_t slash_index = full_jid.find('/');
+  if (slash_index == std::string::npos) {
+    if (bare_jid != nullptr) {

[Sergey Ulanov, 2015/05/15 18:22:37]

nit: don't need "!=nullptr" part

[John Williams, 2015/05/15 21:59:43]

Done.

+      *bare_jid = full_jid;
+    }
+    if (resource_suffix != nullptr) {
+      resource_suffix->clear();
+    }
+    return false;
+  } else {

[Sergey Ulanov, 2015/05/15 18:22:37]

don't need else because the if above always returns.
See last item in
https://www.chromium.org/developers/coding-style#Code_Formatting

[John Williams, 2015/05/15 21:59:43]

Done.

+    if (bare_jid != nullptr) {
+      *bare_jid = std::string(full_jid, 0, slash_index);
+    }
+    if (resource_suffix != nullptr) {
+      *resource_suffix = std::string(full_jid, slash_index);
+    }
+    return true;
+  }
 }
 
 }  // namespace remoting