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

Unified Diff: remoting/protocol/connection_to_host.h

Issue 9005034: Refactor SignalStrategy so that it can be reused for multiple connections. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: - Created 8 years, 12 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
« no previous file with comments | « remoting/jingle_glue/xmpp_signal_strategy.cc ('k') | remoting/protocol/connection_to_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/connection_to_host.h
diff --git a/remoting/protocol/connection_to_host.h b/remoting/protocol/connection_to_host.h
index bba17fd43f6a2c9dc91508a4de6ef8962b49276d..86bd005b52fb169fadab1df2e74a917c9ab13b7b 100644
--- a/remoting/protocol/connection_to_host.h
+++ b/remoting/protocol/connection_to_host.h
@@ -40,7 +40,7 @@ class SessionConfig;
class VideoReader;
class VideoStub;
-class ConnectionToHost : public SignalStrategy::StatusObserver,
+class ConnectionToHost : public SignalStrategy::Listener,
public SessionManager::Listener {
public:
enum State {
@@ -89,12 +89,11 @@ class ConnectionToHost : public SignalStrategy::StatusObserver,
virtual HostStub* host_stub();
// SignalStrategy::StatusObserver interface.
- virtual void OnStateChange(
- SignalStrategy::StatusObserver::State state) OVERRIDE;
- virtual void OnJidChange(const std::string& full_jid) OVERRIDE;
+ virtual void OnSignalStrategyStateChange(
+ SignalStrategy::State state) OVERRIDE;
// SessionManager::Listener interface.
- virtual void OnSessionManagerInitialized() OVERRIDE;
+ virtual void OnSessionManagerReady() OVERRIDE;
virtual void OnIncomingSession(
Session* session,
SessionManager::IncomingSessionResponse* response) OVERRIDE;
@@ -106,10 +105,6 @@ class ConnectionToHost : public SignalStrategy::StatusObserver,
State state() const;
private:
- // Called on the jingle thread after we've successfully to XMPP server. Starts
- // P2P connection to the host.
- void InitSession();
-
// Callback for |session_|.
void OnSessionStateChange(Session::State state);
@@ -143,7 +138,6 @@ class ConnectionToHost : public SignalStrategy::StatusObserver,
VideoStub* video_stub_;
scoped_ptr<SignalStrategy> signal_strategy_;
- std::string local_jid_;
scoped_ptr<SessionManager> session_manager_;
scoped_ptr<Session> session_;
« no previous file with comments | « remoting/jingle_glue/xmpp_signal_strategy.cc ('k') | remoting/protocol/connection_to_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698