Linux pairing registry delegate implementation

remoting/protocol/pairing_registry_unittest.cc

Index: remoting/protocol/pairing_registry_unittest.cc
diff --git a/remoting/protocol/pairing_registry_unittest.cc b/remoting/protocol/pairing_registry_unittest.cc
index fcab43a888007413394054f30718faacdf355035..0093a857dc5465cd25c2b80e65504251bef39be6 100644
--- a/remoting/protocol/pairing_registry_unittest.cc
+++ b/remoting/protocol/pairing_registry_unittest.cc
@@ -21,7 +21,7 @@ class MockDelegate : public PairingRegistry::Delegate {
   // MockDelegate saves to an explicit external PairedClients instance because
   // PairingRegistry takes ownership of it and makes no guarantees about its
   // lifetime, so this is a simple way of getting access to pairing results.
-  MockDelegate(PairingRegistry::PairedClients* paired_clients)
+  explicit MockDelegate(PairingRegistry::PairedClients* paired_clients)
       : paired_clients_(paired_clients) {
   }
 
@@ -29,6 +29,12 @@ class MockDelegate : public PairingRegistry::Delegate {
       const PairingRegistry::PairedClients& paired_clients) OVERRIDE {
     *paired_clients_ = paired_clients;
   }
+  virtual PairingRegistry::PairedClients LoadOnCurrentThread() OVERRIDE {
+    // This convenience method should not be called by the PairingRegistry and
+    // isn't called by the unit test either.
+    EXPECT_TRUE(false);

[Lambros, 2013/05/31 18:51:59]

nit: ADD_FAILURE(); is cleaner.

[Jamie, 2013/06/19 17:56:44]

This is gone in the current version.

+    return PairingRegistry::PairedClients();
+  }
 
  protected:
   PairingRegistry::PairedClients* paired_clients_;
@@ -36,6 +42,7 @@ class MockDelegate : public PairingRegistry::Delegate {
 
 TEST_F(PairingRegistryTest, LoadAndLookup) {
   PairingRegistry::Pairing client_info = {
+    base::Time(),
     "client_id",
     "client_name",
     "shared_secret"