NetEq: Add new method last_output_sample_rate_hz

NetEq: Add new method last_output_sample_rate_hz

This change moves the logics for keeping track of the last ouput
sample rate from AcmReceiver to NetEq, where it fits better. The
getter function AcmReceiver::current_sample_rate_hz() is renamed to
last_output_sample_rate_hz().

BUG=webrtc:3520
Committed: https://crrev.com/d89814bfd78b4c4ab6bf00e6ca4fd9e9ee6055c4
Cr-Commit-Position: refs/heads/master@{#10754}

[hlundin-webrtc, 2015-11-23 11:54:32]

henrik.lundin@webrtc.org changed reviewers:
+ kwiberg@webrtc.org

[hlundin-webrtc, 2015-11-23 11:54:33]

Karl,

Please, take a look.

https://codereview.webrtc.org/1467163002/diff/1/webrtc/modules/audio_coding/m...
File webrtc/modules/audio_coding/main/acm2/acm_receiver.cc (right):

https://codereview.webrtc.org/1467163002/diff/1/webrtc/modules/audio_coding/m...
webrtc/modules/audio_coding/main/acm2/acm_receiver.cc:234: int
samples_per_channel_int = resampler_.Resample10Msec(
The only actual change in this block is that the local current_sample_rate_hz is
used instead of current_sample_rate_hz_.

https://codereview.webrtc.org/1467163002/diff/1/webrtc/modules/audio_coding/m...
webrtc/modules/audio_coding/main/acm2/acm_receiver.cc:250: int
samples_per_channel_int = resampler_.Resample10Msec(
The only actual change in this block is that the local current_sample_rate_hz is
used instead of current_sample_rate_hz_.

[kwiberg-webrtc, 2015-11-23 13:11:51]

lgtm

https://codereview.webrtc.org/1467163002/diff/1/webrtc/modules/audio_coding/m...
File webrtc/modules/audio_coding/main/acm2/acm_receiver.cc (right):

https://codereview.webrtc.org/1467163002/diff/1/webrtc/modules/audio_coding/m...
webrtc/modules/audio_coding/main/acm2/acm_receiver.cc:225: const int
current_sample_rate_hz = neteq_->last_output_sample_rate_hz();
Excellent idea to use const here.

https://codereview.webrtc.org/1467163002/diff/1/webrtc/modules/audio_coding/m...
webrtc/modules/audio_coding/main/acm2/acm_receiver.cc:234: int
samples_per_channel_int = resampler_.Resample10Msec(
On 2015/11/23 11:54:33, hlundin-webrtc wrote:
> The only actual change in this block is that the local current_sample_rate_hz
is
> used instead of current_sample_rate_hz_.

Acknowledged.

https://codereview.webrtc.org/1467163002/diff/1/webrtc/modules/audio_coding/m...
webrtc/modules/audio_coding/main/acm2/acm_receiver.cc:250: int
samples_per_channel_int = resampler_.Resample10Msec(
On 2015/11/23 11:54:33, hlundin-webrtc wrote:
> The only actual change in this block is that the local current_sample_rate_hz
is
> used instead of current_sample_rate_hz_.

Acknowledged.

[hlundin-webrtc, 2015-11-23 13:22:33]

The CQ bit was checked by henrik.lundin@webrtc.org

[commit-bot: I haz the power, 2015-11-23 13:22:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1467163002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1467163002/1

[commit-bot: I haz the power, 2015-11-23 14:49:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-23 14:49:38]

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