Hooking up target audio bitrate to audio network adaptor.

Hooking up target audio bitrate to audio network adaptor.

After the landing of BitrateController, it is time to hook up the network data (target_audio_bitrate_bps) required by BitrateController.

BUG=webrtc:6303
Committed: https://crrev.com/e5e632f8737442add11123826fce0d68f55b0b4c
Cr-Commit-Position: refs/heads/master@{#14406}

[minyue-webrtc, 2016-09-23 07:39:15]

Description was changed from

==========
Hooking up target audio bitrate to audio network adaptor.

BUG=webrtc:6303
==========

to

==========
Hooking up target audio bitrate to audio network adaptor.

After the landing of BitrateController, it is time to hook up the network data
(target_audio_bitrate_bps) required by BitrateController.

BUG=webrtc:6303
==========

[minyue-webrtc, 2016-09-23 07:39:15]

minyue@webrtc.org changed reviewers:
+ henrik.lundin@webrtc.org, michaelt@webrtc.org

[minyue-webrtc, 2016-09-23 07:39:43]

Hi Michael and Henrik,

Please take a look at this small CL. Thanks!

https://codereview.webrtc.org/2364473005/diff/1/webrtc/modules/audio_coding/a...
File
webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.cc
(right):

https://codereview.webrtc.org/2364473005/diff/1/webrtc/modules/audio_coding/a...
webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.cc:62:
void AudioNetworkAdaptorImpl::SetReceiverFrameLengthRange(
moved this method because it was not in the same order as it is in the header.

[hlundin-webrtc, 2016-09-26 15:16:51]

lgtm

https://codereview.webrtc.org/2364473005/diff/1/webrtc/modules/audio_coding/a...
File
webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.cc
(right):

https://codereview.webrtc.org/2364473005/diff/1/webrtc/modules/audio_coding/a...
webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.cc:62:
void AudioNetworkAdaptorImpl::SetReceiverFrameLengthRange(
On 2016/09/23 07:39:43, minyue-webrtc wrote:
> moved this method because it was not in the same order as it is in the header.

Acknowledged.

[michaelt, 2016-09-27 11:53:42]

lgtm

[minyue-webrtc, 2016-09-27 12:02:39]

The CQ bit was checked by minyue@webrtc.org

[commit-bot: I haz the power, 2016-09-27 12:02:43]

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-09-27 14:03:10]

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

[commit-bot: I haz the power, 2016-09-27 14:03:11]

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

[minyue-webrtc, 2016-09-27 14:14:34]

The CQ bit was checked by minyue@webrtc.org

[commit-bot: I haz the power, 2016-09-27 14:14:38]

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-09-27 16:15:12]

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

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

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

[minyue-webrtc, 2016-09-27 19:45:41]

The CQ bit was checked by minyue@webrtc.org

[commit-bot: I haz the power, 2016-09-27 19:45:45]

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-09-27 19:54:21]

Description was changed from

==========
Hooking up target audio bitrate to audio network adaptor.

After the landing of BitrateController, it is time to hook up the network data
(target_audio_bitrate_bps) required by BitrateController.

BUG=webrtc:6303
==========

to

==========
Hooking up target audio bitrate to audio network adaptor.

After the landing of BitrateController, it is time to hook up the network data
(target_audio_bitrate_bps) required by BitrateController.

BUG=webrtc:6303
==========

[commit-bot: I haz the power, 2016-09-27 19:54:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 19:54:30]

Description was changed from

==========
Hooking up target audio bitrate to audio network adaptor.

After the landing of BitrateController, it is time to hook up the network data
(target_audio_bitrate_bps) required by BitrateController.

BUG=webrtc:6303
==========

to

==========
Hooking up target audio bitrate to audio network adaptor.

After the landing of BitrateController, it is time to hook up the network data
(target_audio_bitrate_bps) required by BitrateController.

BUG=webrtc:6303

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

[commit-bot: I haz the power, 2016-09-27 19:54:31]

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