Add encode/decode time tracing to audio_coding.

Add encode/decode time tracing to audio_coding.

Also removes virtual from VideoDecoder::Decode and updated mocks and
tests accordingly to use VideoDecoder::DecodeInternal instead.

BUG=webrtc:5167
R=henrik.lundin@webrtc.org

Committed: https://chromium.googlesource.com/external/webrtc/+/d7b7ae8bdaaf51d885bbbcf6d2ccd2da2522ab03

[pbos-webrtc, 2015-12-08 10:34:04]

PTAL woooo

[hlundin-webrtc, 2015-12-08 10:44:52]

LGTM with one nit, wooooo!

https://codereview.webrtc.org/1512483003/diff/1/webrtc/modules/audio_coding/a...
File webrtc/modules/audio_coding/acm2/audio_coding_module_unittest_oldapi.cc
(right):

https://codereview.webrtc.org/1512483003/diff/1/webrtc/modules/audio_coding/a...
webrtc/modules/audio_coding/acm2/audio_coding_module_unittest_oldapi.cc:979:
class DecodeForwarder {
Comment on the intended use: forward a call from DecodeInternal to Decode.

[pbos-webrtc, 2015-12-08 12:28:24]

add comment

[pbos-webrtc, 2015-12-08 12:28:27]

https://codereview.webrtc.org/1512483003/diff/1/webrtc/modules/audio_coding/a...
File webrtc/modules/audio_coding/acm2/audio_coding_module_unittest_oldapi.cc
(right):

https://codereview.webrtc.org/1512483003/diff/1/webrtc/modules/audio_coding/a...
webrtc/modules/audio_coding/acm2/audio_coding_module_unittest_oldapi.cc:979:
class DecodeForwarder {
On 2015/12/08 10:44:52, hlundin-webrtc wrote:
> Comment on the intended use: forward a call from DecodeInternal to Decode.

Done.

[pbos-webrtc, 2015-12-08 12:28:47]

The CQ bit was checked by pbos@webrtc.org

[pbos-webrtc, 2015-12-08 12:28:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
henrik.lundin@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1512483003/#ps20001 (title:
"add comment")

[commit-bot: I haz the power, 2015-12-08 12:28:55]

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

[commit-bot: I haz the power, 2015-12-08 12:41:51]

Description was changed from

==========
Add encode/decode time tracing to audio_coding.

Also removes virtual from VideoDecoder::Decode and updated mocks and
tests accordingly to use VideoDecoder::DecodeInternal instead.

BUG=webrtc:5167
R=henrik.lundin@webrtc.org
==========

to

==========
Add encode/decode time tracing to audio_coding.

Also removes virtual from VideoDecoder::Decode and updated mocks and
tests accordingly to use VideoDecoder::DecodeInternal instead.

BUG=webrtc:5167
R=henrik.lundin@webrtc.org

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

[commit-bot: I haz the power, 2015-12-08 12:41:51]

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

[pbos-webrtc, 2015-12-08 12:47:10]

Description was changed from

==========
Add encode/decode time tracing to audio_coding.

Also removes virtual from VideoDecoder::Decode and updated mocks and
tests accordingly to use VideoDecoder::DecodeInternal instead.

BUG=webrtc:5167
R=henrik.lundin@webrtc.org

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

to

==========
Add encode/decode time tracing to audio_coding.

Also removes virtual from VideoDecoder::Decode and updated mocks and
tests accordingly to use VideoDecoder::DecodeInternal instead.

BUG=webrtc:5167
R=henrik.lundin@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/d7b7ae8bdaaf51d885bbbcf6d...
==========

[pbos-webrtc, 2015-12-08 12:47:11]

Committed patchset #2 (id:20001) manually as
d7b7ae8bdaaf51d885bbbcf6d2ccd2da2522ab03 (presubmit successful).