Make ModuleRtpRtcpImpl member variables private.

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 dfb31c4661b04e776dfa2318e15f030909fd6187..0e44d4b7447a06de299d4b02567e2c77bbd095ca 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h
+++ b/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h
@@ -318,16 +318,13 @@ class ModuleRtpRtcpImpl : public RtpRtcp, public RTCPReceiver::ModuleRtpRtcp {
 
   const Clock* clock() const { return clock_; }
 
-  // TODO(nisse): Demote all member variables to private, as soon as
-  // downstream code is updated to use the above accessor methods.
+ private:
   RTPSender rtp_sender_;

[danilchap, 2017/03/16 10:16:26]

put members in private section below functions.
i.e. below TimeToSendFullNackList()

[nisse-webrtc, 2017/03/16 10:20:50]

Done.

-
   RTCPSender rtcp_sender_;
   RTCPReceiver rtcp_receiver_;
 
   const Clock* const clock_;
 
- private:
   FRIEND_TEST_ALL_PREFIXES(RtpRtcpImplTest, Rtt);
   FRIEND_TEST_ALL_PREFIXES(RtpRtcpImplTest, RttForReceiverOnly);
   int64_t RtcpReportInterval();