Ping backup connection at a slower rate

talk/app/webrtc/webrtcsession.cc

Index: talk/app/webrtc/webrtcsession.cc
diff --git a/talk/app/webrtc/webrtcsession.cc b/talk/app/webrtc/webrtcsession.cc
index ada35c5e4d7c262a2d64c9d5de8c9f22c8b30417..0cf28dcf7f775baeb940e972e4c3a80a7e00bba5 100644
--- a/talk/app/webrtc/webrtcsession.cc
+++ b/talk/app/webrtc/webrtcsession.cc
@@ -1243,6 +1243,8 @@ cricket::IceConfig WebRtcSession::ParseIceConfig(
     const PeerConnectionInterface::RTCConfiguration& config) const {
   cricket::IceConfig ice_config;
   ice_config.receiving_timeout_ms = config.ice_connection_receiving_timeout;

[pthatcher1, 2015/11/20 05:34:37]

We should have made _ms on all of the places for the receiving timeout as well. 
It looks we only put it in half the places.

[honghaiz3, 2015/11/20 20:10:06]

I sort of did that intentionally because the variable name is already very long.
And I did not add _ms to backup_connection_ping_interval for the same reason.
Let me know if you prefer adding that regardless of the variable length. 

+  ice_config.backup_connection_ping_interval =
+      config.ice_backup_connection_ping_interval;
   ice_config.gather_continually = (config.continual_gathering_policy ==
                                    PeerConnectionInterface::GATHER_CONTINUALLY);
   return ice_config;