1) Fixing TrackAudioRenderer::Render() to interpret the second parameters as delay in frames, not i…

1) Fixing TrackAudioRenderer::Render() to interpret the second parameters as delay in frames, not in milliseconds.
2) Changing output buffer size back to "optimal". It was recently changed to use output buffer size in milliseconds,
but this does not make much sence, because system (well, at least linux) provides default size in frames and does not care about milliseconds.
For example, it can provide default values of 512 for any sample rate.

BUG=596174
Committed: https://crrev.com/8fc0d951347d29e48a2241173383657869cf506e
Cr-Commit-Position: refs/heads/master@{#387600}

[o1ka, 2016-04-15 10:06:46]

olka@chromium.org changed reviewers:
+ tommi@chromium.org

[o1ka, 2016-04-15 10:06:47]

PTAL.

The CL which caused the bug (https://codereview.chromium.org/1687213002)
introduced some other inaccuracies (like, in some places parameters are not
renamed from |delay_milliseconds| to |frames_delay|, which could be misleading
in callback typedefs), but I want to keep this CL as small as possible since we
may want to merge it.

[tommi (sloooow) - chröme, 2016-04-15 13:05:46]

lgtm

[o1ka, 2016-04-15 13:10:45]

The CQ bit was checked by olka@chromium.org

[commit-bot: I haz the power, 2016-04-15 13:10:57]

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

[commit-bot: I haz the power, 2016-04-15 15:04:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-15 15:05:13]

Description was changed from

==========
1) Fixing TrackAudioRenderer::Render() to interpret the second parameters as
delay in frames, not in milliseconds.
2) Changing output buffer size back to "optimal". It was recently changed to use
output buffer size in milliseconds,
but this does not make much sence, because system (well, at least linux)
provides default size in frames and does not care about milliseconds.
For example, it can provide default values of 512 for any sample rate.

BUG=596174
==========

to

==========
1) Fixing TrackAudioRenderer::Render() to interpret the second parameters as
delay in frames, not in milliseconds.
2) Changing output buffer size back to "optimal". It was recently changed to use
output buffer size in milliseconds,
but this does not make much sence, because system (well, at least linux)
provides default size in frames and does not care about milliseconds.
For example, it can provide default values of 512 for any sample rate.

BUG=596174

Committed: https://crrev.com/8fc0d951347d29e48a2241173383657869cf506e
Cr-Commit-Position: refs/heads/master@{#387600}
==========

[commit-bot: I haz the power, 2016-04-15 15:05:14]

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

[o1ka, 2016-05-12 13:41:49]

Description was changed from

==========
1) Fixing TrackAudioRenderer::Render() to interpret the second parameters as
delay in frames, not in milliseconds.
2) Changing output buffer size back to "optimal". It was recently changed to use
output buffer size in milliseconds,
but this does not make much sence, because system (well, at least linux)
provides default size in frames and does not care about milliseconds.
For example, it can provide default values of 512 for any sample rate.

BUG=596174

Committed: https://crrev.com/8fc0d951347d29e48a2241173383657869cf506e
Cr-Commit-Position: refs/heads/master@{#387600}
==========

to

==========
1) Fixing TrackAudioRenderer::Render() to interpret the second parameters as
delay in frames, not in milliseconds.
2) Changing output buffer size back to "optimal". It was recently changed to use
output buffer size in milliseconds,
but this does not make much sence, because system (well, at least linux)
provides default size in frames and does not care about milliseconds.
For example, it can provide default values of 512 for any sample rate.

BUG=596174

Committed: https://crrev.com/8fc0d951347d29e48a2241173383657869cf506e
Cr-Commit-Position: refs/heads/master@{#387600}
==========