Unify the FillAudioEncoderTimeSeries with existing processing functions.

Unify the FillAudioEncoderTimeSeries with existing processing functions.
Use lambdas instead of function objects.

BUG=webrtc:7323
Review-Url: https://codereview.webrtc.org/2743933004
Cr-Commit-Position: refs/heads/master@{#17208}
Committed: https://chromium.googlesource.com/external/webrtc/+/53dc23c28f6a4cc89065e41b6db51a5d81cd7954

[terelius, 2017-03-10 15:00:50]

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

[terelius, 2017-03-10 15:00:51]

Hi Michael!

Mixing function objects and lambdas feel a bit inconsistent to me. I made a CL
based on yours, but using only lambdas. What do you think?

[michaelt, 2017-03-13 09:30:47]

Nice, this is probably the better solution that mine fancy "function objects and
lambdas template one".
lgtm

BTW should we add unit test for this class ?

[terelius, 2017-03-13 11:57:35]

The CQ bit was checked by terelius@webrtc.org

[commit-bot: I haz the power, 2017-03-13 11:57:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[terelius, 2017-03-13 12:04:39]

On 2017/03/13 09:30:47, michaelt wrote:
> Nice, this is probably the better solution that mine fancy "function objects
and
> lambdas template one".
> lgtm
> 
> BTW should we add unit test for this class ?

Yes, I agree that we should add unit tests. The reason we haven't done it
already is that it isn't clear what exactly should be tested. Altering labels,
plot style or rounding during intermediate representations (e.g. float vs
double) should preferably not make the test fail. Maybe start with something
simple just to verify that the tool builds and produces output?

[commit-bot: I haz the power, 2017-03-13 12:24:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489406255529970, "parent_rev":
"39e1289e6480f0a8c8d9b8f9f63a5ca255209093", "commit_rev":
"53dc23c28f6a4cc89065e41b6db51a5d81cd7954"}

[commit-bot: I haz the power, 2017-03-13 12:24:11]

Description was changed from

==========
Unify the FillAudioEncoderTimeSeries with existing processing functions.
Use lambdas instead of function objects.

BUG=webrtc:7323
==========

to

==========
Unify the FillAudioEncoderTimeSeries with existing processing functions.
Use lambdas instead of function objects.

BUG=webrtc:7323

Review-Url: https://codereview.webrtc.org/2743933004
Cr-Commit-Position: refs/heads/master@{#17208}
Committed:
https://chromium.googlesource.com/external/webrtc/+/53dc23c28f6a4cc89065e41b6...
==========

[commit-bot: I haz the power, 2017-03-13 12:24:12]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/53dc23c28f6a4cc89065e41b6...