Use case-insensitive comparison when comparing JIDs.

remoting/host/access_verifier.cc

Index: remoting/host/access_verifier.cc
diff --git a/remoting/host/access_verifier.cc b/remoting/host/access_verifier.cc
index 62718162c7f315a9d7c623d870a3f755bddf0958..19397d63072e62a6a8ba7934e6627c86b2b9a525 100644
--- a/remoting/host/access_verifier.cc
+++ b/remoting/host/access_verifier.cc
@@ -35,8 +35,9 @@ bool AccessVerifier::VerifyPermissions(
     const std::string& encoded_access_token) {
   CHECK(initialized_);
   // Check that the client has the same bare jid as the host, i.e.
-  // client's full jid starts with host's bare jid.
-  if (!StartsWithASCII(client_jid, host_jid_prefix_, true)) {
+  // client's full JID starts with host's bare jid. Comparison is case
+  // insensitive.
+  if (!StartsWithASCII(client_jid, host_jid_prefix_, false)) {

[awong, 2011/02/14 20:25:54]

I'm thinking that we should preface this with an IsStringASCII check.

Reading through the jingle docs, it seems like they might not always be ascii if
we're outside of the google talk network (eg., federated XMPP user).

[Sergey Ulanov, 2011/02/14 21:38:20]

Added IsStringASCII() verification, not sure if it matters though:
StartsWithASCII() should ignore case only for ASCII characters.

[awong, 2011/02/14 21:59:06]

I'm not 100% sure it does... It falls down to strncasecmp which I would expect
understands locale and can do all sorts of fun things.

[Sergey Ulanov, 2011/02/14 22:05:36]

Yes, you are right. Thanks for catching this!

     return false;
   }