Wire up RTP keep-alive in ortc api.

webrtc/ortc/rtptransportcontrolleradapter.h

 /*
  *  Copyright 2017 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 59 matching lines @@
   ~RtpTransportControllerAdapter() override;
 
   // RtpTransportControllerInterface implementation.
   std::vector<RtpTransportInterface*> GetTransports() const override;
 
   // These methods are used by OrtcFactory to create RtpTransports, RtpSenders
   // and RtpReceivers using this controller. Called "CreateProxied" because
   // these methods return proxies that will safely call methods on the correct
   // thread.
   RTCErrorOr<std::unique_ptr<RtpTransportInterface>> CreateProxiedRtpTransport(
-      const RtcpParameters& rtcp_parameters,
+      const RtpTransportParameters& rtcp_parameters,
       PacketTransportInterface* rtp,
       PacketTransportInterface* rtcp);
 
   RTCErrorOr<std::unique_ptr<SrtpTransportInterface>>
-  CreateProxiedSrtpTransport(const RtcpParameters& rtcp_parameters,
+  CreateProxiedSrtpTransport(const RtpTransportParameters& rtcp_parameters,
                              PacketTransportInterface* rtp,
                              PacketTransportInterface* rtcp);
 
   // |transport_proxy| needs to be a proxy to a transport because the
   // application may call GetTransport() on the returned sender or receiver,
   // and expects it to return a thread-safe transport proxy.
   RTCErrorOr<std::unique_ptr<OrtcRtpSenderInterface>> CreateProxiedRtpSender(
       cricket::MediaType kind,
       RtpTransportInterface* transport_proxy);
   RTCErrorOr<std::unique_ptr<OrtcRtpReceiverInterface>>
   CreateProxiedRtpReceiver(cricket::MediaType kind,
                            RtpTransportInterface* transport_proxy);
 
   // Methods used internally by other "adapter" classes.
   rtc::Thread* signaling_thread() const { return signaling_thread_; }
   rtc::Thread* worker_thread() const { return worker_thread_; }
 
   RTCError SetRtcpParameters(const RtcpParameters& parameters,
                              RtpTransportInterface* inner_transport);
+  RTCError SetRtpTransportParameters(const RtpTransportParameters& parameters);

[Taylor Brandstetter, 2017/07/17 23:13:11]

Do we need two methods here or just one?

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

I think one method is enough here. Removed SetRtcpParameters and moved it's code
to the SetRtpTransportParameters.

 
   cricket::VoiceChannel* voice_channel() { return voice_channel_; }
   cricket::VideoChannel* video_channel() { return video_channel_; }
 
   // |primary_ssrc| out parameter is filled with either
   // |parameters.encodings[0].ssrc|, or a generated SSRC if that's left unset.
   RTCError ValidateAndApplyAudioSenderParameters(
       const RtpParameters& parameters,
       uint32_t* primary_ssrc);
   RTCError ValidateAndApplyVideoSenderParameters(
@@ skipping 95 matching lines @@
   bool have_video_sender_ = false;
   bool have_audio_receiver_ = false;
   bool have_video_receiver_ = false;
 
   RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(RtpTransportControllerAdapter);
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_ORTC_RTPTRANSPORTCONTROLLERADAPTER_H_