Refactor SignalStrategy so that it can be reused for multiple connections.

remoting/protocol/session_manager.h

Index: remoting/protocol/session_manager.h
diff --git a/remoting/protocol/session_manager.h b/remoting/protocol/session_manager.h
index ad484fa606b48c423a1fba8bb67fd8535b5edad7..53901bd953fc374c85dc614c0b73f9ccee3f8d16 100644
--- a/remoting/protocol/session_manager.h
+++ b/remoting/protocol/session_manager.h
@@ -89,11 +89,10 @@ class SessionManager : public base::NonThreadSafe {
     Listener() { }
     ~Listener() { }
 
-    // Called when the session manager has finished
-    // initialization. May be called from Init() or after Init()
-    // returns. Outgoing connections can be created after this method
-    // is called.
-    virtual void OnSessionManagerInitialized() = 0;
+    // Called when the session manager is ready. May be called from

[Wez, 2011/12/21 23:35:30]

nit: ready for what? Perhaps just merge this sentence with the one about
outgoing connections? Can we also accept incoming connections after this
notification?

[Sergey Ulanov, 2011/12/22 21:45:10]

Done
OnIncomingSession() may be called even before this method is called.

+    // Init() or after Init() returns. Outgoing connections can be
+    // created after this method is called.
+    virtual void OnSessionManagerReady() = 0;
 
     // Called when a new session is received. If the host decides to
     // accept the session it should set the |response| to
@@ -117,8 +116,7 @@ class SessionManager : public base::NonThreadSafe {
   // will also take ownership of these objects. On the client side
   // pass in NULL for |private_key| and empty string for
   // |certificate|.
-  virtual void Init(const std::string& local_jid,
-                    SignalStrategy* signal_strategy,
+  virtual void Init(SignalStrategy* signal_strategy,
                     Listener* listener,
                     bool allow_nat_traversal) = 0;