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

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 9 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
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..86a72207273456aeed1d924438603742789dbc50 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,13 @@ 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;
+ virtual bool OnSignalStrategyIncomingStanza(
+ const buzz::XmlElement* stanza) OVERRIDE;
// SessionManager::Listener interface.
- virtual void OnSessionManagerInitialized() OVERRIDE;
+ virtual void OnSessionManagerReady() OVERRIDE;
virtual void OnIncomingSession(
Session* session,
SessionManager::IncomingSessionResponse* response) OVERRIDE;
@@ -106,10 +107,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 +140,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_;

Powered by Google App Engine
This is Rietveld 408576698