Remove V1 authenticator support from Me2Me host.

remoting/protocol/me2me_host_authenticator_factory.cc

 // Copyright (c) 2012 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/protocol/me2me_host_authenticator_factory.h"
 
 #include "base/base64.h"
 #include "base/string_util.h"
 #include "crypto/rsa_private_key.h"
 #include "remoting/protocol/channel_authenticator.h"
@@ skipping 89 matching lines @@
   // Verify that the client's jid is an ASCII string, and then check
   // that the client has the same bare jid as the host, i.e. client's
   // full JID starts with host's bare jid. Comparison is case
   // insensitive.
   if (!IsStringASCII(remote_jid) ||
       !StartsWithASCII(remote_jid, local_jid_prefix_, false)) {
     LOG(ERROR) << "Rejecting incoming connection from " << remote_jid;
     return scoped_ptr<Authenticator>(new RejectingAuthenticator());
   }
 
-  if (NegotiatingAuthenticator::IsNegotiableMessage(first_message)) {
-    return NegotiatingAuthenticator::CreateForHost(
-        local_cert_, *local_private_key_, shared_secret_hash_.value,
-        shared_secret_hash_.hash_function);
-  }
-
-  // TODO(sergeyu): Old clients still use V1 auth protocol. Remove
-  // this once we are done migrating to V2. crbug.com/110483 .
-  return scoped_ptr<Authenticator>(new V1HostAuthenticator(
-      local_cert_, *local_private_key_, "", remote_jid));
+  return NegotiatingAuthenticator::CreateForHost(
+      local_cert_, *local_private_key_, shared_secret_hash_.value,
+      shared_secret_hash_.hash_function);

[Jamie, 2012/02/06 21:03:45]

I take it that IsNegotiableMessage was being used as a test for v1 vs. v2?
That's not obvious from the name, but if that's the case, lgtm.

 }
 
 }  // namespace protocol
 }  // namespace remoting