Log connection type to syslogs and to the server.

remoting/host/host_event_logger_win.cc

Index: remoting/host/host_event_logger_win.cc
diff --git a/remoting/host/host_event_logger_win.cc b/remoting/host/host_event_logger_win.cc
index b0dd3c058d1dcdd3c4532b37b3047430791c845e..60bab3dcf5f5579f1aa9ccc6fc75d16926a7c2e0 100644
--- a/remoting/host/host_event_logger_win.cc
+++ b/remoting/host/host_event_logger_win.cc
@@ -15,6 +15,7 @@
 #include "net/base/ip_endpoint.h"
 #include "remoting/host/chromoting_host.h"
 #include "remoting/host/host_status_observer.h"
+#include "remoting/protocol/transport.h"
 
 #include "remoting_host_messages.h"
 
@@ -37,8 +38,7 @@ class HostEventLoggerWin : public HostEventLogger, public HostStatusObserver {
   virtual void OnClientRouteChange(
       const std::string& jid,
       const std::string& channel_name,
-      const net::IPEndPoint& remote_end_point,
-      const net::IPEndPoint& local_end_point) OVERRIDE;
+      const protocol::TransportRoute& route) OVERRIDE;
   virtual void OnShutdown() OVERRIDE;
 
  private:
@@ -56,7 +56,7 @@ class HostEventLoggerWin : public HostEventLogger, public HostStatusObserver {
 } //namespace
 
 HostEventLoggerWin::HostEventLoggerWin(ChromotingHost* host,
-                                           const std::string& application_name)
+                                       const std::string& application_name)
     : host_(host),
       event_log_(NULL) {
   event_log_ = RegisterEventSourceW(NULL,
@@ -91,13 +91,13 @@ void HostEventLoggerWin::OnAccessDenied(const std::string& jid) {
 void HostEventLoggerWin::OnClientRouteChange(
     const std::string& jid,
     const std::string& channel_name,
-    const net::IPEndPoint& remote_end_point,
-    const net::IPEndPoint& local_end_point) {
+    const protocol::TransportRoute& route) {
   std::vector<string16> strings(4);
   strings[0] = ASCIIToUTF16(jid);
-  strings[1] = ASCIIToUTF16(remote_end_point.ToString());
-  strings[2] = ASCIIToUTF16(local_end_point.ToString());
+  strings[1] = ASCIIToUTF16(route.remote_end_point.ToString());
+  strings[2] = ASCIIToUTF16(route.local_end_point.ToString());
   strings[3] = ASCIIToUTF16(channel_name);
+  strings[4] = ASCIIToUTF16(route.GetTypeString());
   Log(EVENTLOG_INFORMATION_TYPE, MSG_HOST_CLIENT_ROUTING_CHANGED, strings);

[alexeypa (please no reviews), 2012/03/19 22:04:59]

This requires updating remoting/host/remoting_host_messages.mc file.

Note that normally you would add another message leaving
MSG_HOST_CLIENT_ROUTING_CHANGED intact. This would be needed to make sure that
already logged messages could be shown properly after the host binary is
updated. In this case however, the binary wasn't released yet, so you can patch
the existing message.

[Sergey Ulanov, 2012/03/19 23:21:03]

Thanks for noticing it!

 
Whoa, that sucks.

 }