PairingAuthenticator implementation and plumbing.

remoting/host/remoting_me2me_host.cc

Index: remoting/host/remoting_me2me_host.cc
diff --git a/remoting/host/remoting_me2me_host.cc b/remoting/host/remoting_me2me_host.cc
index 6ee42b9ff8d15d1e8cc16b47949b21f3d2e84cd5..7d72bbd02f29f0b59f969a156481cfd11c2edd5f 100644
--- a/remoting/host/remoting_me2me_host.cc
+++ b/remoting/host/remoting_me2me_host.cc
@@ -67,6 +67,7 @@
 #include "remoting/host/usage_stats_consent.h"
 #include "remoting/jingle_glue/xmpp_signal_strategy.h"
 #include "remoting/protocol/me2me_host_authenticator_factory.h"
+#include "remoting/protocol/pairing_registry.h"
 
 #if defined(OS_POSIX)
 #include <signal.h>
@@ -466,8 +467,12 @@ void HostProcess::CreateAuthenticatorFactory() {
   scoped_ptr<protocol::AuthenticatorFactory> factory;
 
   if (token_url_.is_empty() && token_validation_url_.is_empty()) {
+    // TODO(jamiewalch): Add a proper pairing registry here once all the code
+    // is committed.
+    scoped_refptr<remoting::protocol::PairingRegistry> pairing_registry;
     factory = protocol::Me2MeHostAuthenticatorFactory::CreateWithSharedSecret(
-        local_certificate, key_pair_, host_secret_hash_);
+        local_certificate, key_pair_, host_secret_hash_, pairing_registry);
+

[Sergey Ulanov, 2013/05/17 01:57:17]

nit: remove this line.

   } else if (token_url_.is_valid() && token_validation_url_.is_valid()) {
     scoped_ptr<protocol::ThirdPartyHostAuthenticator::TokenValidatorFactory>
         token_validator_factory(new TokenValidatorFactoryImpl(
@@ -475,6 +480,7 @@ void HostProcess::CreateAuthenticatorFactory() {
             context_->url_request_context_getter()));
     factory = protocol::Me2MeHostAuthenticatorFactory::CreateWithThirdPartyAuth(
         local_certificate, key_pair_, token_validator_factory.Pass());
+

[Sergey Ulanov, 2013/05/17 01:57:17]

nit: remove this line.

[Jamie, 2013/05/21 01:24:33]

I think it's more readable with these branches separated out a bit. If you feel
strongly that it doesn't, then I'll remove the blank lines, but I'd prefer to
leave them.

   } else {
     // TODO(rmsousa): If the policy is bad the host should not go online. It
     // should keep running, but not connected, until the policies are fixed.