Make ModuleRtpRtcpImpl member variables private.

Make ModuleRtpRtcpImpl member variables private.

This is a follow-up to https://codereview.webrtc.org/2747743002.

BUG=webrtc:7135
Review-Url: https://codereview.webrtc.org/2750283003
Cr-Commit-Position: refs/heads/master@{#17271}
Committed: https://chromium.googlesource.com/external/webrtc/+/150708e061b88c1e7c02aba13d4196a4913f99cc

[nisse-webrtc, 2017-03-16 08:46:06]

nisse@webrtc.org changed reviewers:
+ danilchap@webrtc.org, stefan@webrtc.org

[nisse-webrtc, 2017-03-16 08:46:07]

PTAL. I hope this is landable as soon as danil's application update is landed.

[stefan-webrtc, 2017-03-16 08:47:55]

lgtm

[danilchap, 2017-03-16 10:16:27]

lgtm

https://codereview.webrtc.org/2750283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
File webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h (right):

https://codereview.webrtc.org/2750283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h:322: RTPSender rtp_sender_;
put members in private section below functions.
i.e. below TimeToSendFullNackList()

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

https://codereview.webrtc.org/2750283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
File webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h (right):

https://codereview.webrtc.org/2750283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.h:322: RTPSender rtp_sender_;
On 2017/03/16 10:16:26, danilchap wrote:
> put members in private section below functions.
> i.e. below TimeToSendFullNackList()

Done.

[nisse-webrtc, 2017-03-16 10:45:38]

The CQ bit was checked by nisse@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-16 10:45:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-03-16 11:08:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-16 11:08:11]

Dry run: This issue passed the CQ dry run.

[nisse-webrtc, 2017-03-16 12:00:51]

The CQ bit was checked by nisse@webrtc.org

[nisse-webrtc, 2017-03-16 12:00:52]

The patchset sent to the CQ was uploaded after l-g-t-m from
danilchap@webrtc.org, stefan@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2750283003/#ps20001 (title:
"Fix ordering in private section.")

[commit-bot: I haz the power, 2017-03-16 12:00:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-03-16 12:02:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489665651688480,
"parent_rev": "9c192b2b06cb012b3f515923aeb0a3a0f9a09025", "commit_rev":
"150708e061b88c1e7c02aba13d4196a4913f99cc"}

[commit-bot: I haz the power, 2017-03-16 12:02:56]

Description was changed from

==========
Make ModuleRtpRtcpImpl member variables private.

This is a follow-up to https://codereview.webrtc.org/2747743002.

BUG=webrtc:7135
==========

to

==========
Make ModuleRtpRtcpImpl member variables private.

This is a follow-up to https://codereview.webrtc.org/2747743002.

BUG=webrtc:7135

Review-Url: https://codereview.webrtc.org/2750283003
Cr-Commit-Position: refs/heads/master@{#17271}
Committed:
https://chromium.googlesource.com/external/webrtc/+/150708e061b88c1e7c02aba13...
==========

[commit-bot: I haz the power, 2017-03-16 12:02:57]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/150708e061b88c1e7c02aba13...