Minor cleanups in JingleSession and JingleSessionManager.

remoting/protocol/jingle_session_manager.h

Index: remoting/protocol/jingle_session_manager.h
diff --git a/remoting/protocol/jingle_session_manager.h b/remoting/protocol/jingle_session_manager.h
index e629f40e1601e4acb21bc1766bc42a9825c84974..9fe57e90fda75091f0b955907f75092cc4bcb9db 100644
--- a/remoting/protocol/jingle_session_manager.h
+++ b/remoting/protocol/jingle_session_manager.h
@@ -37,11 +37,13 @@ class JingleSessionManager
     : public SessionManager,
       public cricket::SessionClient {
  public:
-  JingleSessionManager(
+  virtual ~JingleSessionManager();
+
+  static JingleSessionManager* Create();
+  static JingleSessionManager* Create(

[Wez, 2011/07/06 21:24:25]

nit: It's not clear from the name what kinds of network manager, etc, Connect()
causes to be used; give it a more descriptive name?

[Sergey Ulanov, 2011/07/06 23:15:01]

Done.

       talk_base::NetworkManager* network_manager,
       talk_base::PacketSocketFactory* socket_factory,
       PortAllocatorSessionFactory* port_allocator_session_factory);
-  virtual ~JingleSessionManager();
 
   // SessionManager interface.
   virtual void Init(const std::string& local_jid,
@@ -76,6 +78,11 @@ class JingleSessionManager
  private:
   friend class JingleSession;
 
+  JingleSessionManager(
+      talk_base::NetworkManager* network_manager,
+      talk_base::PacketSocketFactory* socket_factory,
+      PortAllocatorSessionFactory* port_allocator_session_factory);
+
   // Called by JingleSession when a new connection is
   // initiated. Returns true if session is accepted.
   bool AcceptConnection(JingleSession* jingle_session,
@@ -84,13 +91,6 @@ class JingleSessionManager
   // Called by JingleSession when it is being destroyed.
   void SessionDestroyed(JingleSession* jingle_session);
 
-  void DoConnect(
-      JingleSession* jingle_session,
-      const std::string& host_jid,
-      const std::string& host_public_key,
-      const std::string& client_token,
-      Session::StateChangeCallback* state_change_callback);
-
   // Callback for JingleInfoRequest.
   void OnJingleInfo(
       const std::string& token,