Split RtcpReceiver::HandleSenderReceiverReport into two functions

webrtc/modules/rtp_rtcp/source/rtcp_receiver.h

Index: webrtc/modules/rtp_rtcp/source/rtcp_receiver.h
diff --git a/webrtc/modules/rtp_rtcp/source/rtcp_receiver.h b/webrtc/modules/rtp_rtcp/source/rtcp_receiver.h
index ace82f043d6e4d533426bc7030cd8a3a6a046ddd..21e4fb92090269b8e312ab2a6159287b83faf493 100644
--- a/webrtc/modules/rtp_rtcp/source/rtcp_receiver.h
+++ b/webrtc/modules/rtp_rtcp/source/rtcp_receiver.h
@@ -132,7 +132,12 @@ class RTCPReceiver {
       uint32_t remoteSSRC);
   RTCPHelp::RTCPReceiveInformation* GetReceiveInformation(uint32_t remoteSSRC);
 
-  void HandleSenderReceiverReport(
+  void HandleSenderReport(
+      RTCPUtility::RTCPParserV2& rtcpParser,

[philipel, 2016/09/15 11:58:19]

Mark arguments as const or change them to pointers.

[danilchap, 2016/09/15 12:17:30]

Acknowledged.
In one or two CLs this particular argument will be replaced with a different one
(sure it will follow style, will be const&).
Draft is linked from this CL.
Changing it in this CL would generate too many offtopic changes.

+      RTCPHelp::RTCPPacketInformation& rtcpPacketInformation)
+      EXCLUSIVE_LOCKS_REQUIRED(_criticalSectionRTCPReceiver);
+
+  void HandleReceiverReport(
       RTCPUtility::RTCPParserV2& rtcpParser,

[philipel, 2016/09/15 11:58:19]

Ditto

[danilchap, 2016/09/15 12:17:30]

Acknowledged.

       RTCPHelp::RTCPPacketInformation& rtcpPacketInformation)
       EXCLUSIVE_LOCKS_REQUIRED(_criticalSectionRTCPReceiver);