Merge RtcpReceive::Handle<Packet>Item functions into Handle<Packet>

Merge RtcpReceiver::Handle<Packet>Item functions into Handle<Packet>
As a preparation to replace parsing implementation.

BUG=webrtc:5260
R=philipel@webrtc.org

Committed: https://crrev.com/530b3f5d06b873ca6ad72207b680260ed92afd44
Cr-Commit-Position: refs/heads/master@{#14240}

[danilchap, 2016-09-14 11:40:15]

Description was changed from

==========
Merge in RtcpReceiver Handle<Packet>Item functions into Handle<Packet>
As a preparation to replace parsing implementation.

BUG=webrtc:5260
==========

to

==========
Merge RtcpReceiver::Handle<Packet>Item functions into Handle<Packet>
As a preparation to replace parsing implementation.

BUG=webrtc:5260
==========

[danilchap, 2016-09-14 11:46:03]

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

[danilchap, 2016-09-14 11:46:04]

To make https://codereview.webrtc.org/2316093002/ more reviewable slicing out
this change

[philipel, 2016-09-15 15:30:00]

lgtm with a few nits

https://codereview.webrtc.org/2337283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
File webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc (right):

https://codereview.webrtc.org/2337283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc:728: assert(cnameInfo);
RTC_CHECK

https://codereview.webrtc.org/2337283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc:734: if (stats_callback_ !=
NULL) {
nullptr

[danilchap, 2016-09-15 16:31:47]

https://codereview.webrtc.org/2337283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
File webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc (right):

https://codereview.webrtc.org/2337283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc:728: assert(cnameInfo);
On 2016/09/15 15:30:00, philipel wrote:
> RTC_CHECK

Done.
RTC_DCHECK

https://codereview.webrtc.org/2337283003/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtcp_receiver.cc:734: if (stats_callback_ !=
NULL) {
On 2016/09/15 15:30:00, philipel wrote:
> nullptr

Done.
recommended actually just if(stats_callback_)

[commit-bot: I haz the power, 2016-09-15 16:41:15]

Description was changed from

==========
Merge RtcpReceiver::Handle<Packet>Item functions into Handle<Packet>
As a preparation to replace parsing implementation.

BUG=webrtc:5260
==========

to

==========
Merge RtcpReceiver::Handle<Packet>Item functions into Handle<Packet>
As a preparation to replace parsing implementation.

BUG=webrtc:5260
R=philipel@webrtc.org

Committed: https://crrev.com/530b3f5d06b873ca6ad72207b680260ed92afd44
Cr-Commit-Position: refs/heads/master@{#14240}
==========

[danilchap, 2016-09-15 16:41:16]

Description was changed from

==========
Merge RtcpReceiver::Handle<Packet>Item functions into Handle<Packet>
As a preparation to replace parsing implementation.

BUG=webrtc:5260
R=philipel@webrtc.org

Committed: https://crrev.com/530b3f5d06b873ca6ad72207b680260ed92afd44
Cr-Commit-Position: refs/heads/master@{#14240}
==========

to

==========
Merge RtcpReceiver::Handle<Packet>Item functions into Handle<Packet>
As a preparation to replace parsing implementation.

BUG=webrtc:5260
R=philipel@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/530b3f5d06b873ca6ad72207b...
==========

[commit-bot: I haz the power, 2016-09-15 16:41:17]

Patchset 2 (id:??) landed as
https://crrev.com/530b3f5d06b873ca6ad72207b680260ed92afd44
Cr-Commit-Position: refs/heads/master@{#14240}

[danilchap, 2016-09-15 16:41:17]

Committed patchset #2 (id:20001) manually as
530b3f5d06b873ca6ad72207b680260ed92afd44 (presubmit successful).