Move RTCPHelp::RTCPReportBlockInformation into RTCPReceiver

Move RTCPHelp::RTCPReportBlockInformation into RTCPReceiver
removing RTCPHelp namespace and rtcp_receiver_help files,
cleaning style of the ReportBlockInformation usage.

BUG=webrtc:5565
Committed: https://crrev.com/28b03eb4493d759ad1b84cc6df8e2992ba18f02e
Cr-Commit-Position: refs/heads/master@{#14527}

[danilchap, 2016-10-03 11:22:19]

danilchap@webrtc.org changed reviewers:
+ philipel@webrtc.org

[danilchap, 2016-10-03 11:22:20]

Mostly style change.
Better rtt calculation still planned in a different dedicated CL.

[philipel, 2016-10-03 12:33:17]

lg

https://codereview.webrtc.org/2390643002/diff/20001/webrtc/modules/rtp_rtcp/s...
File webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc (right):

https://codereview.webrtc.org/2390643002/diff/20001/webrtc/modules/rtp_rtcp/s...
webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc:90: struct
RTCPReceiver::ReportBlockInformation {
WDYT about |AggregatedRttInfo| instead of |ReportBlockInformation|?

[danilchap, 2016-10-03 16:10:12]

https://codereview.webrtc.org/2390643002/diff/20001/webrtc/modules/rtp_rtcp/s...
File webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc (right):

https://codereview.webrtc.org/2390643002/diff/20001/webrtc/modules/rtp_rtcp/s...
webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc:90: struct
RTCPReceiver::ReportBlockInformation {
On 2016/10/03 12:33:17, philipel wrote:
> WDYT about |AggregatedRttInfo| instead of |ReportBlockInformation|?

The structure still hold the ReportBlock itself.
(It was also holding max jitter that I removed as unused.)

how about |ReportBlockWithRtt|?
or you would prefer longer name: |ReportBlockWithAggregatedRtt|

[philipel, 2016-10-05 08:04:28]

lgtm

[danilchap, 2016-10-05 08:13:16]

The CQ bit was checked by danilchap@webrtc.org

[commit-bot: I haz the power, 2016-10-05 08:13:23]

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, 2016-10-05 08:28:52]

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

[commit-bot: I haz the power, 2016-10-05 08:28:53]

Try jobs failed on following builders:
  win_clang_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_clang_dbg/builds/7013)

[danilchap, 2016-10-05 13:13:24]

The CQ bit was checked by danilchap@webrtc.org

[commit-bot: I haz the power, 2016-10-05 13:13:37]

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, 2016-10-05 13:59:48]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-05 13:59:53]

Description was changed from

==========
Move RTCPHelp::RTCPReportBlockInformation into RTCPReceiver
removing RTCPHelp namespace and rtcp_receiver_help files,
cleaning style of the ReportBlockInformation usage.

BUG=webrtc:5565
==========

to

==========
Move RTCPHelp::RTCPReportBlockInformation into RTCPReceiver
removing RTCPHelp namespace and rtcp_receiver_help files,
cleaning style of the ReportBlockInformation usage.

BUG=webrtc:5565

Committed: https://crrev.com/28b03eb4493d759ad1b84cc6df8e2992ba18f02e
Cr-Commit-Position: refs/heads/master@{#14527}
==========

[commit-bot: I haz the power, 2016-10-05 13:59:54]

Patchset 4 (id:??) landed as
https://crrev.com/28b03eb4493d759ad1b84cc6df8e2992ba18f02e
Cr-Commit-Position: refs/heads/master@{#14527}