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

Unified Diff: remoting/test/test_chromoting_client.h

Issue 1520323007: Simplify ConnectionToHost interface. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@sm_cleanup
Patch Set: Created 5 years 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
« no previous file with comments | « remoting/protocol/fake_connection_to_host.cc ('k') | remoting/test/test_chromoting_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/test/test_chromoting_client.h
diff --git a/remoting/test/test_chromoting_client.h b/remoting/test/test_chromoting_client.h
index cb0c730b0837483ab3eb038ef100011107c3f84a..ce9f9ac5cbb04980d7fcf501b5c9373e0e4fb976 100644
--- a/remoting/test/test_chromoting_client.h
+++ b/remoting/test/test_chromoting_client.h
@@ -67,7 +67,8 @@ class TestChromotingClient : public ClientUserInterface,
// Unregisters an observerer from notifications for remote connection events.
void RemoveRemoteConnectionObserver(RemoteConnectionObserver* observer);
- // Used to set a fake/mock connection object for TestChromotingClient tests.
+ // Used to set a fake/mock dependencies for tests.
+ void SetSignalStrategyForTests(scoped_ptr<SignalStrategy> signal_strategy);
void SetConnectionToHostForTests(
scoped_ptr<protocol::ConnectionToHost> connection_to_host);
@@ -114,8 +115,8 @@ class TestChromotingClient : public ClientUserInterface,
// Processes video packets from the host.
scoped_ptr<VideoRenderer> video_renderer_;
- // Used to establish an XMPP connection with the google talk service.
- scoped_ptr<XmppSignalStrategy> signal_strategy_;
+ // SignalStrategy used for connection signaling.
+ scoped_ptr<SignalStrategy> signal_strategy_;
DISALLOW_COPY_AND_ASSIGN(TestChromotingClient);
};
« no previous file with comments | « remoting/protocol/fake_connection_to_host.cc ('k') | remoting/test/test_chromoting_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698