Pull out common code from client and host versions of LogToServer

remoting/host/log_to_server_host_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/message_loop/message_loop.h"
 #include "base/message_loop/message_loop_proxy.h"
 #include "remoting/host/host_status_monitor_fake.h"
-#include "remoting/host/log_to_server.h"
+#include "remoting/host/log_to_server_host.h"
 #include "remoting/jingle_glue/mock_objects.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gmock_mutant.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/libjingle/source/talk/xmllite/xmlelement.h"
 
 using buzz::XmlElement;
 using buzz::QName;
 using testing::_;
 using testing::DeleteArg;
@@ skipping 95 matching lines @@
     return false;
   }
   if (log_entry->NextChild()) {
     return false;
   }
   return true;
 }
 
 }  // namespace
 
-class LogToServerTest : public testing::Test {
+class LogToServerHostTest : public testing::Test {
  public:
-  LogToServerTest() {}
+  LogToServerHostTest() {}
   virtual void SetUp() OVERRIDE {
     message_loop_proxy_ = base::MessageLoopProxy::current();
     EXPECT_CALL(signal_strategy_, AddListener(_));
     log_to_server_.reset(
-        new LogToServer(host_status_monitor_.AsWeakPtr(),
-                        ServerLogEntry::ME2ME,
-                        &signal_strategy_,
-                        kTestBotJid));
+        new LogToServerHost(host_status_monitor_.AsWeakPtr(),
+                            ServerLogEntry::ME2ME,
+                            &signal_strategy_,
+                            kTestBotJid));
     EXPECT_CALL(signal_strategy_, RemoveListener(_));
   }
 
  protected:
   base::MessageLoop message_loop_;
   scoped_refptr<base::MessageLoopProxy> message_loop_proxy_;
   MockSignalStrategy signal_strategy_;
-  scoped_ptr<LogToServer> log_to_server_;
+  scoped_ptr<LogToServerHost> log_to_server_;
   HostStatusMonitorFake host_status_monitor_;
 };
 
-TEST_F(LogToServerTest, SendNow) {
+TEST_F(LogToServerHostTest, SendNow) {
   {
     InSequence s;
     EXPECT_CALL(signal_strategy_, GetLocalJid())
         .WillRepeatedly(Return(kHostJid));
     EXPECT_CALL(signal_strategy_, AddListener(_));
     EXPECT_CALL(signal_strategy_, GetNextId());
     EXPECT_CALL(signal_strategy_, SendStanzaPtr(IsClientConnected("direct")))
         .WillOnce(DoAll(DeleteArg<0>(), Return(true)));
     EXPECT_CALL(signal_strategy_, RemoveListener(_))
         .WillOnce(QuitMainMessageLoop(&message_loop_))
         .RetiresOnSaturation();
   }
   log_to_server_->OnSignalStrategyStateChange(SignalStrategy::CONNECTED);
   protocol::TransportRoute route;
   route.type = protocol::TransportRoute::DIRECT;
   log_to_server_->OnClientRouteChange(kClientJid1, "video", route);
   log_to_server_->OnClientAuthenticated(kClientJid1);
   log_to_server_->OnClientConnected(kClientJid1);
   log_to_server_->OnSignalStrategyStateChange(SignalStrategy::DISCONNECTED);
   message_loop_.Run();
 }
 
-TEST_F(LogToServerTest, SendLater) {
+TEST_F(LogToServerHostTest, SendLater) {
   protocol::TransportRoute route;
   route.type = protocol::TransportRoute::DIRECT;
   log_to_server_->OnClientRouteChange(kClientJid1, "video", route);
   log_to_server_->OnClientAuthenticated(kClientJid1);
   log_to_server_->OnClientConnected(kClientJid1);
   {
     InSequence s;
     EXPECT_CALL(signal_strategy_, GetLocalJid())
         .WillRepeatedly(Return(kHostJid));
     EXPECT_CALL(signal_strategy_, AddListener(_));
     EXPECT_CALL(signal_strategy_, GetNextId());
     EXPECT_CALL(signal_strategy_, SendStanzaPtr(IsClientConnected("direct")))
         .WillOnce(DoAll(DeleteArg<0>(), Return(true)));
     EXPECT_CALL(signal_strategy_, RemoveListener(_))
         .WillOnce(QuitMainMessageLoop(&message_loop_))
         .RetiresOnSaturation();
   }
   log_to_server_->OnSignalStrategyStateChange(SignalStrategy::CONNECTED);
   log_to_server_->OnSignalStrategyStateChange(SignalStrategy::DISCONNECTED);
   message_loop_.Run();
 }
 
-TEST_F(LogToServerTest, SendTwoEntriesLater) {
+TEST_F(LogToServerHostTest, SendTwoEntriesLater) {
   protocol::TransportRoute route1;
   route1.type = protocol::TransportRoute::DIRECT;
   log_to_server_->OnClientRouteChange(kClientJid1, "video", route1);
   log_to_server_->OnClientAuthenticated(kClientJid1);
   log_to_server_->OnClientConnected(kClientJid1);
   protocol::TransportRoute route2;
   route2.type = protocol::TransportRoute::STUN;
   log_to_server_->OnClientRouteChange(kClientJid2, "video", route2);
   log_to_server_->OnClientAuthenticated(kClientJid2);
   log_to_server_->OnClientConnected(kClientJid2);
   {
     InSequence s;
     EXPECT_CALL(signal_strategy_, GetLocalJid())
         .WillRepeatedly(Return(kHostJid));
     EXPECT_CALL(signal_strategy_, AddListener(_));
     EXPECT_CALL(signal_strategy_, GetNextId());
     EXPECT_CALL(signal_strategy_, SendStanzaPtr(
             IsTwoClientsConnected("direct", "stun")))
         .WillOnce(DoAll(DeleteArg<0>(), Return(true)));
     EXPECT_CALL(signal_strategy_, RemoveListener(_))
         .WillOnce(QuitMainMessageLoop(&message_loop_))
         .RetiresOnSaturation();
   }
   log_to_server_->OnSignalStrategyStateChange(SignalStrategy::CONNECTED);
   log_to_server_->OnSignalStrategyStateChange(SignalStrategy::DISCONNECTED);
   message_loop_.Run();
 }
 
-TEST_F(LogToServerTest, HandleRouteChangeInUnusualOrder) {
+TEST_F(LogToServerHostTest, HandleRouteChangeInUnusualOrder) {
   {
     InSequence s;
     EXPECT_CALL(signal_strategy_, GetLocalJid())
         .WillRepeatedly(Return(kHostJid));
     EXPECT_CALL(signal_strategy_, AddListener(_));
     EXPECT_CALL(signal_strategy_, GetNextId());
     EXPECT_CALL(signal_strategy_, SendStanzaPtr(IsClientConnected("direct")))
         .WillOnce(DoAll(DeleteArg<0>(), Return(true)));
     EXPECT_CALL(signal_strategy_, GetNextId());
     EXPECT_CALL(signal_strategy_, SendStanzaPtr(IsClientDisconnected()))
@@ skipping 15 matching lines @@
   route2.type = protocol::TransportRoute::STUN;
   log_to_server_->OnClientRouteChange(kClientJid2, "video", route2);
   log_to_server_->OnClientDisconnected(kClientJid1);
   log_to_server_->OnClientAuthenticated(kClientJid2);
   log_to_server_->OnClientConnected(kClientJid2);
   log_to_server_->OnSignalStrategyStateChange(SignalStrategy::DISCONNECTED);
   message_loop_.Run();
 }
 
 }  // namespace remoting