Reland Remove the deprecated EncodeInternal interface from AudioEncoder

Reland Remove the deprecated EncodeInternal interface from AudioEncoder

Remove the deprecated EncodeInternal interface from AudioEncoder

Also hid MaxEncodedBytes by making it private. It will get removed as soon as subclasses have had time to remove their overrides.

BUG=webrtc:5591
Committed: https://crrev.com/2903ba5ff3cc1aa8fa58aa3fc0f4de53840d97e1
Cr-Commit-Position: refs/heads/master@{#12409}

[ossu, 2016-04-15 12:17:13]

Description was changed from

==========
Reland Remove the deprecated EncodeInternal interface from AudioEncoder

Remove the deprecated EncodeInternal interface from AudioEncoder

Also hid MaxEncodedBytes by making it private. It will get removed as soon as
subclasses have had time to remove their overrides.

BUG=webrtc:5591
==========

to

==========
Reland Remove the deprecated EncodeInternal interface from AudioEncoder

Remove the deprecated EncodeInternal interface from AudioEncoder

Also hid MaxEncodedBytes by making it private. It will get removed as soon as
subclasses have had time to remove their overrides.

BUG=webrtc:5591
==========

[ossu, 2016-04-15 12:17:13]

ossu@webrtc.org changed reviewers:
+ kwiberg@webrtc.org

[ossu, 2016-04-15 12:18:50]

Created a new CL for re-landing removing EncodeInternal after dependencies have
been fixed.

[kwiberg-webrtc, 2016-04-15 13:18:06]

lgtm

https://codereview.webrtc.org/1881003003/diff/20001/webrtc/modules/audio_codi...
File webrtc/modules/audio_coding/codecs/audio_encoder.h (right):

https://codereview.webrtc.org/1881003003/diff/20001/webrtc/modules/audio_codi...
webrtc/modules/audio_coding/codecs/audio_encoder.h:141: // as these subclasses
have been given a chance to change.
Bug reference?

[ossu, 2016-04-18 13:08:07]

The CQ bit was checked by ossu@webrtc.org

[commit-bot: I haz the power, 2016-04-18 13:08:09]

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

[commit-bot: I haz the power, 2016-04-18 13:14:36]

Description was changed from

==========
Reland Remove the deprecated EncodeInternal interface from AudioEncoder

Remove the deprecated EncodeInternal interface from AudioEncoder

Also hid MaxEncodedBytes by making it private. It will get removed as soon as
subclasses have had time to remove their overrides.

BUG=webrtc:5591
==========

to

==========
Reland Remove the deprecated EncodeInternal interface from AudioEncoder

Remove the deprecated EncodeInternal interface from AudioEncoder

Also hid MaxEncodedBytes by making it private. It will get removed as soon as
subclasses have had time to remove their overrides.

BUG=webrtc:5591
==========

[commit-bot: I haz the power, 2016-04-18 13:14:37]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-18 13:14:48]

Description was changed from

==========
Reland Remove the deprecated EncodeInternal interface from AudioEncoder

Remove the deprecated EncodeInternal interface from AudioEncoder

Also hid MaxEncodedBytes by making it private. It will get removed as soon as
subclasses have had time to remove their overrides.

BUG=webrtc:5591
==========

to

==========
Reland Remove the deprecated EncodeInternal interface from AudioEncoder

Remove the deprecated EncodeInternal interface from AudioEncoder

Also hid MaxEncodedBytes by making it private. It will get removed as soon as
subclasses have had time to remove their overrides.

BUG=webrtc:5591

Committed: https://crrev.com/2903ba5ff3cc1aa8fa58aa3fc0f4de53840d97e1
Cr-Commit-Position: refs/heads/master@{#12409}
==========

[commit-bot: I haz the power, 2016-04-18 13:14:49]

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