Adding packet overhead to audio network adaptor.

Adding packet overhead to audio network adaptor.

BUG=webrtc:6303, webrtc:6762
Committed: https://crrev.com/c9e80eee514a3db7b66f49c02e62e3d5b1f89a9d
Cr-Commit-Position: refs/heads/master@{#15305}

[minyue-webrtc, 2016-11-23 21:38:16]

Description was changed from

==========
finished.


removing audio_encoder changes


audio_encoder


reformat


fixing


Merge branch 'master' into on_overheadsize_changed_1


adding overhead.


half way

BUG=
==========

to

==========
Adding packet overhead to audio network adaptor.

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

[minyue-webrtc, 2016-11-23 21:38:16]

minyue@webrtc.org changed reviewers:
+ michaelt@webrtc.org

[minyue-webrtc, 2016-11-23 21:39:57]

Hi Michael,

Would you take a look at this CL?

[minyue-webrtc, 2016-11-24 08:38:00]

Description was changed from

==========
Adding packet overhead to audio network adaptor.

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

to

==========
Adding packet overhead to audio network adaptor.

BUG=webrtc:6303, webrtc:6762
==========

[minyue-webrtc, 2016-11-24 08:38:00]

minyue@webrtc.org changed reviewers:
+ henrikg@webrtc.org

[minyue-webrtc, 2016-11-24 08:39:09]

minyue@webrtc.org changed reviewers:
+ henrik.lundin@webrtc.org
- henrikg@webrtc.org

[minyue-webrtc, 2016-11-24 08:39:09]

+Henrik,

Michael has seen this CL. PTAL also.

[hlundin-webrtc, 2016-11-24 08:49:01]

lgtm

[minyue-webrtc, 2016-11-24 09:03:47]

Thanks!

Will do one more change when another CL is landed, and then submit this one.

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

https://codereview.webrtc.org/2530653003/diff/1/webrtc/modules/audio_coding/a...
webrtc/modules/audio_coding/audio_network_adaptor/bitrate_controller.cc:40: //
TODO(minyue): DCHECK that field trial that indicates that
Will add this check according to https://codereview.webrtc.org/2517243005/

[michaelt, 2016-11-24 10:35:18]

lgtm

[minyue-webrtc, 2016-11-25 11:34:39]

I added field trial according to https://codereview.webrtc.org/2517243005/

[minyue-webrtc, 2016-11-28 21:28:57]

On 2016/11/25 11:34:39, minyue-webrtc wrote:
> I added field trial according to https://codereview.webrtc.org/2517243005/

Given https://codereview.webrtc.org/2517243005/ has landed. I am committing this
one now.

[minyue-webrtc, 2016-11-29 20:29:11]

The CQ bit was checked by minyue@webrtc.org

[minyue-webrtc, 2016-11-29 20:29:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
henrik.lundin@webrtc.org, michaelt@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2530653003/#ps20001 (title:
"adding field trial")

[commit-bot: I haz the power, 2016-11-29 20:29:17]

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-11-29 21:00:29]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480451351214740,
"parent_rev": "e66f8d8801a8fb0d7a88584771c40ba32f29aa43", "commit_rev":
"d3cf06968eeca6abece27026c29b713582c4a8b3"}

[commit-bot: I haz the power, 2016-11-29 21:00:31]

Description was changed from

==========
Adding packet overhead to audio network adaptor.

BUG=webrtc:6303, webrtc:6762
==========

to

==========
Adding packet overhead to audio network adaptor.

BUG=webrtc:6303, webrtc:6762
==========

[commit-bot: I haz the power, 2016-11-29 21:00:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-29 21:00:41]

Description was changed from

==========
Adding packet overhead to audio network adaptor.

BUG=webrtc:6303, webrtc:6762
==========

to

==========
Adding packet overhead to audio network adaptor.

BUG=webrtc:6303, webrtc:6762

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

[commit-bot: I haz the power, 2016-11-29 21:00:43]

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