Chromium Code Reviews| Index: remoting/protocol/pepper_session_manager.h | 
| diff --git a/remoting/protocol/pepper_session_manager.h b/remoting/protocol/pepper_session_manager.h | 
| index 508c161a1081e5074f54850d7919ae014d84e350..0e9a41f27c01c0bf5a98136431976c3e02515471 100644 | 
| --- a/remoting/protocol/pepper_session_manager.h | 
| +++ b/remoting/protocol/pepper_session_manager.h | 
| @@ -12,8 +12,8 @@ | 
| #include "base/memory/ref_counted.h" | 
| #include "net/base/x509_certificate.h" | 
| #include "remoting/jingle_glue/signal_strategy.h" | 
| -#include "remoting/protocol/pepper_channel.h" | 
| #include "remoting/protocol/session_manager.h" | 
| +#include "remoting/protocol/transport.h" | 
| #include "remoting/protocol/transport_config.h" | 
| namespace pp { | 
| @@ -44,7 +44,8 @@ class PepperSession; | 
| class PepperSessionManager : public SessionManager, | 
| public SignalStrategy::Listener { | 
| public: | 
| - explicit PepperSessionManager(pp::Instance* pp_instance); | 
| + explicit PepperSessionManager( | 
| + scoped_ptr<TransportFactory> transport_factory); | 
| 
 
Wez
2012/02/07 01:17:37
Why have the TransportFactory passed in here, give
 
Sergey Ulanov
2012/02/07 04:41:12
Yes. The idea is to make this class generic and no
 
 | 
| virtual ~PepperSessionManager(); | 
| // SessionManager interface. | 
| @@ -83,7 +84,7 @@ class PepperSessionManager : public SessionManager, | 
| // Called by PepperSession when it is being destroyed. | 
| void SessionDestroyed(PepperSession* session); | 
| - pp::Instance* pp_instance_; | 
| + scoped_ptr<TransportFactory> transport_factory_; | 
| SignalStrategy* signal_strategy_; | 
| scoped_ptr<AuthenticatorFactory> authenticator_factory_; |