Add accessor functions for protected member variables of ModuleRtpRtcpImpl.

webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h

Index: webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h b/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h
index 10639bbca38b57956802a039b6e862bf678d7c27..2b80bff2c1552001ab68b73df7548e2d5065db7f 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h
+++ b/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h
@@ -312,6 +312,19 @@ class ModuleRtpRtcpImpl : public RtpRtcp, public RTCPReceiver::ModuleRtpRtcp {
  protected:
   bool UpdateRTCPReceiveInformationTimers();
 
+  RTPSender* rtp_sender() { return &rtp_sender_; }
+  const RTPSender* rtp_sender() const { return &rtp_sender_; }
+
+  RTCPSender* rtcp_sender() { return &rtcp_sender_; }
+  const RTCPSender* rtcp_sender() const { return &rtcp_sender_; }
+
+  RTCPReceiver* rtcp_receiver() { return &rtcp_receiver_; }
+  const RTCPReceiver* rtcp_receiver() const { return &rtcp_receiver_; }
+
+  Clock* clock() { return clock_; }

[danilchap, 2017/03/13 12:57:19]

since Clock interface has only const functions, it might be better to make this
accessor const:
const Clock* clock() const { return clock_; }

[nisse-webrtc, 2017/03/13 14:05:46]

Done. Made the |clock_| member const const as well.

+
+  // TODO(nisse): Demote all member variables to private, as soon as
+  // downstream code is updated to use the above accessor methods.
   RTPSender rtp_sender_;
 
   RTCPSender rtcp_sender_;