[Chromoting] Add platform data to heartbeats.

remoting/host/server_log_entry.h

Index: remoting/host/server_log_entry.h
diff --git a/remoting/host/server_log_entry.h b/remoting/host/server_log_entry.h
index 8e5be839740fe347d7ecfcd37905382f9bb6b5fa..64b8d5e1dc89bc25a8b7f2b49f9c80c435fcaa9b 100644
--- a/remoting/host/server_log_entry.h
+++ b/remoting/host/server_log_entry.h
@@ -25,10 +25,18 @@ class ServerLogEntry {
     ME2ME
   };
 
+  // Constructs a log stanza. The caller should add one or more log entry
+  // stanzas as children of this stanza, before sending the log stanza to
+  // the remoting bot.
+  static scoped_ptr<buzz::XmlElement> MakeLogStanza();

[Wez, 2012/05/18 01:27:14]

nit: "Log" is a little redundant in the name; how about MakeStanza(), or
NewStanza() to distinguish this method from the Make* methods below?

[simonmorris, 2012/05/18 17:06:57]

Done.

+
   // Constructs a log entry for a session state change.
   // Currently this is either connection or disconnection.
   static ServerLogEntry* MakeSessionStateChange(bool connection);
 
+  // Constructs a log entry for a heartbeat.
+  static ServerLogEntry* MakeHeartbeat();

[Wez, 2012/05/18 01:27:14]

MakeHeartbeat unfortunately reads in HeartbeatSender as though this method
creates the heart-beat; consider ServerLogEntry::MakeForHeartbeat() or
ForHeartbeat()?

[simonmorris, 2012/05/18 17:06:57]

Done.

+
   ~ServerLogEntry();
 
   // Adds fields describing the host to this log entry.