Wire up RTP keep-alive in ortc api.

webrtc/pc/rtptransport.cc

Index: webrtc/pc/rtptransport.cc
diff --git a/webrtc/pc/rtptransport.cc b/webrtc/pc/rtptransport.cc
index 6b0141fc8e92986e2ec690360f3a97805cf17ee6..7a2df3b4316ff6ddbad9601b615301c33d4dab8b 100644
--- a/webrtc/pc/rtptransport.cc
+++ b/webrtc/pc/rtptransport.cc
@@ -135,6 +135,23 @@ RtcpParameters RtpTransport::GetRtcpParameters() const {
   return rtcp_parameters_;
 }
 
+RTCError RtpTransport::SetRtpTransportParameters(
+    const RtpTransportParameters& parameters) {
+  if (rtp_transport_parameters_.keepalive.timeout_interval_ms > 0 &&

[Taylor Brandstetter, 2017/07/12 16:19:50]

The parameters in this class don't appear to be used. So it may be better to
leave this method as a TODO and just return an error, so people don't wrongly
assume things are wired up already.

Note that this class is currently only used in the PeerConnection object model
(hence it being in "pc/"), but it's what we plan to eventually use for both the
ORTC and PeerConnection code paths, replacing "RtpTransportAdapter" with it.

[ilnik, 2017/07/21 12:21:19]

Acknowledged.

+      rtp_transport_parameters_.keepalive != parameters.keepalive) {
+    LOG_AND_RETURN_ERROR(
+        RTCErrorType::INVALID_STATE,
+        "Cannot change RTP keep-alive parameters once enabled.");
+  }
+
+  rtp_transport_parameters_ = parameters;
+  return RTCError::OK();
+}
+
+RtpTransportParameters RtpTransport::GetRtpTransportParameters() const {
+  return rtp_transport_parameters_;
+}
+
 RtpTransportAdapter* RtpTransport::GetInternal() {
   return nullptr;
 }