Pull out common code from client and host versions of LogToServer

remoting/jingle_glue/log_to_server.h

Index: remoting/jingle_glue/log_to_server.h
diff --git a/remoting/client/log_to_server.h b/remoting/jingle_glue/log_to_server.h
similarity index 54%
rename from remoting/client/log_to_server.h
rename to remoting/jingle_glue/log_to_server.h
index cd70ff538469c1047c6fc5edc2d6d748565cbfdb..94d29d655a593cd114e9126c51f2285b9a62b755 100644
--- a/remoting/client/log_to_server.h
+++ b/remoting/jingle_glue/log_to_server.h
@@ -2,19 +2,16 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef REMOTING_CLIENT_LOG_TO_SERVER_H_
-#define REMOTING_CLIENT_LOG_TO_SERVER_H_
+#ifndef REMOTING_JINGLE_GLUE_LOG_TO_SERVER_H_
+#define REMOTING_JINGLE_GLUE_LOG_TO_SERVER_H_
 
 #include <deque>
 #include <map>
 #include <string>
 
 #include "base/threading/non_thread_safe.h"
-#include "base/time/time.h"
 #include "remoting/jingle_glue/server_log_entry.h"
 #include "remoting/jingle_glue/signal_strategy.h"
-#include "remoting/protocol/connection_to_host.h"
-#include "remoting/protocol/errors.h"
 
 namespace buzz {
 class XmlElement;
@@ -22,15 +19,8 @@ class XmlElement;
 
 namespace remoting {
 
-class ChromotingStats;
 class IqSender;
 
-// Temporary namespace to prevent conflict with the same-named class in
-// remoting/host when linking unittests.
-//
-// TODO(lambroslambrou): Remove this and factor out any shared code.
-namespace client {
-
 // LogToServer sends log entries to a server.
 // The contents of the log entries are described in server_log_entry.cc.
 // They do not contain any personally identifiable information.
@@ -40,12 +30,7 @@ class LogToServer : public base::NonThreadSafe,
   LogToServer(ServerLogEntry::Mode mode,
               SignalStrategy* signal_strategy,
               const std::string& directory_bot_jid);
-  virtual ~LogToServer();
-
-  // Logs a session state change.
-  void LogSessionStateChange(protocol::ConnectionToHost::State state,
-                             protocol::ErrorCode error);
-  void LogStatistics(remoting::ChromotingStats* statistics);
+  ~LogToServer();
 
   // SignalStrategy::Listener interface.
   virtual void OnSignalStrategyStateChange(
@@ -53,15 +38,12 @@ class LogToServer : public base::NonThreadSafe,
   virtual bool OnSignalStrategyIncomingStanza(
       const buzz::XmlElement* stanza) OVERRIDE;
 
- private:
   void Log(const ServerLogEntry& entry);
-  void SendPendingEntries();
 
-  // Generates a new random session ID.
-  void GenerateSessionId();
+  ServerLogEntry::Mode mode() { return mode_; }
 
-  // Expire the session ID if the maximum duration has been exceeded.
-  void MaybeExpireSessionId();
+ private:
+  void SendPendingEntries();
 
   ServerLogEntry::Mode mode_;
   SignalStrategy* signal_strategy_;
@@ -70,21 +52,9 @@ class LogToServer : public base::NonThreadSafe,
 
   std::deque<ServerLogEntry> pending_entries_;
 
-  // A randomly generated session ID to be attached to log messages. This
-  // is regenerated at the start of a new session.
-  std::string session_id_;
-
-  // Start time of the session.
-  base::TimeTicks session_start_time_;
-
-  // Time when the session ID was generated.
-  base::TimeTicks session_id_generation_time_;
-
   DISALLOW_COPY_AND_ASSIGN(LogToServer);
 };
 
-}  // namespace client
-
 }  // namespace remoting
 
-#endif  // REMOTING_CLIENT_LOG_TO_SERVER_H_
+#endif  // REMOTING_JINGLE_GLUE_LOG_TO_SERVER_H_