Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(515)

Unified Diff: remoting/protocol/pairing_registry_unittest.cc

Issue 15709005: Linux pairing registry delegate implementation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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);
+ 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"

Powered by Google App Engine
This is Rietveld 408576698