Make ModuleRtpRtcpImpl member variables private.

webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h

 /*
  *  Copyright (c) 2012 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 300 matching lines @@
   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_; }
 
   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();
   void SetRtcpReceiverSsrcs(uint32_t main_ssrc);
 
   void set_rtt_ms(int64_t rtt_ms);
   int64_t rtt_ms() const;
 
   bool TimeToSendFullNackList(int64_t now) const;
 
@@ skipping 18 matching lines @@
   PacketLossStats receive_loss_stats_;
 
   // The processed RTT from RtcpRttStats.
   rtc::CriticalSection critical_section_rtt_;
   int64_t rtt_ms_;
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_RTP_RTCP_SOURCE_RTP_RTCP_IMPL_H_