Enable WebRTC support in the remoting host.

remoting/host/host_status_logger.cc

 // Copyright 2014 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 "remoting/host/host_status_logger.h"
 
 #include "base/bind.h"
 #include "remoting/base/constants.h"
 #include "remoting/host/host_status_monitor.h"
 #include "remoting/host/server_log_entry_host.h"
@@ skipping 18 matching lines @@
 
 void HostStatusLogger::LogSessionStateChange(const std::string& jid,
                                              bool connected) {
   DCHECK(CalledOnValidThread());
 
   scoped_ptr<ServerLogEntry> entry(
       MakeLogEntryForSessionStateChange(connected));
   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());
 }
 
 void HostStatusLogger::OnClientConnected(const std::string& jid) {
   DCHECK(CalledOnValidThread());
   LogSessionStateChange(jid, true);
 }
 
 void HostStatusLogger::OnClientDisconnected(const std::string& jid) {
   DCHECK(CalledOnValidThread());
@@ skipping 10 matching lines @@
   if (channel_name == kVideoChannelName) {
     connection_route_type_[jid] = route.type;
   }
 }
 
 void HostStatusLogger::SetSignalingStateForTest(SignalStrategy::State state) {
   log_to_server_.OnSignalStrategyStateChange(state);
 }
 
 }  // namespace remoting