Enable WebRTC support in the remoting host.

remoting/host/host_status_logger.cc

Index: remoting/host/host_status_logger.cc
diff --git a/remoting/host/host_status_logger.cc b/remoting/host/host_status_logger.cc
index d2407eeb8b0d94f16296fee4344a32881991f0ee..eeae93b5b1cae440b982ba862ada4cb918db9cfa 100644
--- a/remoting/host/host_status_logger.cc
+++ b/remoting/host/host_status_logger.cc
@@ -36,10 +36,9 @@ void HostStatusLogger::LogSessionStateChange(const std::string& jid,
   AddHostFieldsToLogEntry(entry.get());
   entry->AddModeField(log_to_server_.mode());
 
-  if (connected) {
-    DCHECK_EQ(connection_route_type_.count(jid), 1u);
+  if (connected && connection_route_type_.count(jid) > 0)

[Jamie, 2015/12/10 18:34:15]

Why this change? And why not just change the DCHECK to consider the >1 case?

[Sergey Ulanov, 2015/12/10 22:18:17]

Currently WebrtcTransport doesn't report route type so when it's being used
connection_route_type_.count(jid) returns 0 and the DCHECK was failing. 

[Jamie, 2015/12/10 22:29:17]

Do you mean that with WebRTC LogSessionStateChange is called more often, and
connected_ might be true before there are any connection route types? Why the
change from ==1 to >0?

[Sergey Ulanov, 2015/12/10 22:48:05]

No. With WebrtcTransport LogSessionStateChange() is never called so
connection_route_type_ is empty. Connection type cannot be reported in that
case.

     AddConnectionTypeToLogEntry(entry.get(), connection_route_type_[jid]);

[Jamie, 2015/12/10 22:29:17]

Is this line still correct, even if connection_route_type_.count(jid) > 1?

[Sergey Ulanov, 2015/12/10 22:48:05]

connection_route_type_ is a map (not std::multimap) so count() can return only 0
or 1.

-  }
+
   log_to_server_.Log(*entry.get());
 }