Revert "Fix crashes in ChromotingHost"

Reverting the patch since it exposed several memory leaks and threading
problems.

TBR=thakis

BUG=70935
TEST=None

Review URL: http://codereview.chromium.org/6266023

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72679 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 6 insertions, 35 deletions

diff --git a/remoting/protocol/connection_to_client.h b/remoting/protocol/connection_to_client.h
index b7aa052..749ef5c 100644
--- a/remoting/protocol/connection_to_client.h
+++ b/remoting/protocol/connection_to_client.h
@@ -55,10 +55,10 @@ class ConnectionToClient :
 
   virtual ~ConnectionToClient();
 
-  virtual void Init(Session* session);
+  virtual void Init(protocol::Session* session);
 
   // Returns the connection in use.
-  virtual Session* session();
+  virtual protocol::Session* session();
 
   // Disconnect the client connection. This method is allowed to be called
   // more than once and calls after the first one will be ignored.
@@ -78,15 +78,15 @@ class ConnectionToClient :
 
  private:
   // Callback for protocol Session.
-  void OnSessionStateChange(Session::State state);
+  void OnSessionStateChange(protocol::Session::State state);
 
   // Process a libjingle state change event on the |loop_|.
-  void StateChangeTask(Session::State state);
+  void StateChangeTask(protocol::Session::State state);
 
   void OnClosed();
 
   // The libjingle channel used to send and receive data from the remote client.
-  scoped_refptr<Session> session_;
+  scoped_refptr<protocol::Session> session_;
 
   scoped_ptr<VideoWriter> video_writer_;
 
diff --git a/remoting/protocol/mock_objects.h b/remoting/protocol/mock_objects.h
index 99826c2..045a414 100644
--- a/remoting/protocol/mock_objects.h
+++ b/remoting/protocol/mock_objects.h
@@ -6,11 +6,9 @@
 #define REMOTING_PROTOCOL_MOCK_OBJECTS_H_
 
 #include "remoting/proto/internal.pb.h"
-#include "remoting/protocol/client_stub.h"
 #include "remoting/protocol/connection_to_client.h"
 #include "remoting/protocol/host_stub.h"
 #include "remoting/protocol/input_stub.h"
-#include "remoting/protocol/session.h"
 #include "remoting/protocol/video_stub.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
@@ -23,10 +21,8 @@ class MockConnectionToClient : public ConnectionToClient {
  public:
   MockConnectionToClient() {}
 
-  MOCK_METHOD1(Init, void(Session* session));
+  MOCK_METHOD1(Init, void(ChromotocolConnection* connection));
   MOCK_METHOD0(video_stub, VideoStub*());
-  MOCK_METHOD0(client_stub, ClientStub*());
-  MOCK_METHOD0(session, Session*());
   MOCK_METHOD0(Disconnect, void());
 
  private:
@@ -82,31 +78,6 @@ class MockVideoStub : public VideoStub {
   DISALLOW_COPY_AND_ASSIGN(MockVideoStub);
 };
 
-class MockSession : public Session {
- public:
-  MockSession() {}
-
-  MOCK_METHOD1(SetStateChangeCallback, void(StateChangeCallback* callback));
-  MOCK_METHOD0(control_channel, net::Socket*());
-  MOCK_METHOD0(event_channel, net::Socket*());
-  MOCK_METHOD0(video_channel, net::Socket*());
-  MOCK_METHOD0(video_rtp_channel, net::Socket*());
-  MOCK_METHOD0(video_rtcp_channel, net::Socket*());
-  MOCK_METHOD0(jid, const std::string&());
-  MOCK_METHOD0(message_loop, MessageLoop*());
-  MOCK_METHOD0(candidate_config, const CandidateSessionConfig*());
-  MOCK_METHOD0(config, const SessionConfig*());
-  MOCK_METHOD1(set_config, void(const SessionConfig* config));
-  MOCK_METHOD0(initiator_token, const std::string&());
-  MOCK_METHOD1(set_initiator_token, void(const std::string& initiator_token));
-  MOCK_METHOD0(receiver_token, const std::string&());
-  MOCK_METHOD1(set_receiver_token, void(const std::string& receiver_token));
-  MOCK_METHOD1(Close, void(Task* closed_task));
-
- private:
-  DISALLOW_COPY_AND_ASSIGN(MockSession);
-};
-
 }  // namespace protocol
 }  // namespace remoting