Remove chain of methods in RtpRtcp module to get current payload frequency for RTCP SRs

Remove chain of methods in RtpRtcp module to get current payload frequency for RTCP SRs, which anyway was stuck to defaults for video/audio.

BUG=webrtc:2795, webrtc:6458
Committed: https://crrev.com/b19d288c94014b1b4167771846573871255a875b
Cr-Commit-Position: refs/heads/master@{#14476}

[the sun, 2016-09-23 19:47:43]

solenberg@webrtc.org changed reviewers:
+ stefan@webrtc.org

[the sun, 2016-09-23 19:47:44]

Take a look at this. Current behavior is maintained. Question is - does it need
a proper fix? Audio RTCP sender reports always use an 8k time base for their rtp
time stamps...

[the sun, 2016-09-26 18:55:31]

Description was changed from

==========
Remove chain of methods in RtpRtcp module to get current payload frequency for
RTCP SRs, which anyway was stuck to defaults for video/audio.

BUG=webrtc:2795
==========

to

==========
Remove chain of methods in RtpRtcp module to get current payload frequency for
RTCP SRs, which anyway was stuck to defaults for video/audio.

BUG=webrtc:2795
==========

[the sun, 2016-09-28 07:57:00]

ping...

[danilchap, 2016-09-28 08:19:34]

On 2016/09/28 07:57:00, the sun wrote:
> ping...

lgtm

I think proper fix is desired. At least a bug should be created.
Wrong frequency means mapping between rtp and ntp time is slightly wrong for
audio stream.
The error is small and doesn't accumulate, so the effect on audio-video sync is
unnoticeable,
still not zero.

Current chain of method most likely will not be used for proper fix, so agree
better to remove them.
for same reason FeedbackState::frequency_hz probably make sense to remove too 
and for now use two constants directly in the RTCPSender::BuildSR (RTCPSender
has bool audio_)

[stefan-webrtc, 2016-09-28 10:22:52]

lgtm, but perhaps follow up with a fix?

[the sun, 2016-09-30 18:09:43]

Description was changed from

==========
Remove chain of methods in RtpRtcp module to get current payload frequency for
RTCP SRs, which anyway was stuck to defaults for video/audio.

BUG=webrtc:2795
==========

to

==========
Remove chain of methods in RtpRtcp module to get current payload frequency for
RTCP SRs, which anyway was stuck to defaults for video/audio.

BUG=webrtc:2795,webrtc:6458
==========

[the sun, 2016-10-03 08:02:26]

On 2016/09/28 10:22:52, stefan-webrtc (holmer) wrote:
> lgtm, but perhaps follow up with a fix?

I've created an issue for this... issue: webrtc:6458. There will be a fix but
not in the short term.

[the sun, 2016-10-03 08:02:31]

The CQ bit was checked by solenberg@webrtc.org

[the sun, 2016-10-03 08:02:31]

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/2362373002/#ps80001 (title:
"comment")

[commit-bot: I haz the power, 2016-10-03 08:02:41]

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-03 10:03:15]

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

[commit-bot: I haz the power, 2016-10-03 10:03:16]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[the sun, 2016-10-03 12:58:58]

The CQ bit was checked by solenberg@webrtc.org

[commit-bot: I haz the power, 2016-10-03 12:59:01]

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-03 13:22:27]

Description was changed from

==========
Remove chain of methods in RtpRtcp module to get current payload frequency for
RTCP SRs, which anyway was stuck to defaults for video/audio.

BUG=webrtc:2795,webrtc:6458
==========

to

==========
Remove chain of methods in RtpRtcp module to get current payload frequency for
RTCP SRs, which anyway was stuck to defaults for video/audio.

BUG=webrtc:2795,webrtc:6458
==========

[commit-bot: I haz the power, 2016-10-03 13:22:28]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-10-03 13:22:35]

Description was changed from

==========
Remove chain of methods in RtpRtcp module to get current payload frequency for
RTCP SRs, which anyway was stuck to defaults for video/audio.

BUG=webrtc:2795,webrtc:6458
==========

to

==========
Remove chain of methods in RtpRtcp module to get current payload frequency for
RTCP SRs, which anyway was stuck to defaults for video/audio.

BUG=webrtc:2795,webrtc:6458

Committed: https://crrev.com/b19d288c94014b1b4167771846573871255a875b
Cr-Commit-Position: refs/heads/master@{#14476}
==========

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

Patchset 5 (id:??) landed as
https://crrev.com/b19d288c94014b1b4167771846573871255a875b
Cr-Commit-Position: refs/heads/master@{#14476}