ppapi: examples: audio_encode: reduce array buffer reallocations

ppapi: examples: audio_encode: prevent buffer reallocations

Prevent buffer reallocations by using an array of buffers to save encoded
audio data in and just create a buffer to contain all of them when the user
wants to save the data to a file.

BUG=570837
Committed: https://crrev.com/f0c88d2d5e63301459685e84f724da4163b32c4d
Cr-Commit-Position: refs/heads/master@{#370940}

[llandwerlin-old, 2016-01-20 15:51:48]

Description was changed from

==========
ppapi: examples: audio_encode: reduce array buffer reallocations

Prevent buffer reallocations by increasing the size exponentially. This
limits the CPU consumption by avoiding reallocation & copies for each
encoded audio buffer.

BUG=570837
==========

to

==========
ppapi: examples: audio_encode: prevent buffer reallocations

Prevent buffer reallocations by using an array of buffers to save encoded
audio data in and just create a buffer to contain all of them when the user
wants to save the data to a file.


BUG=570837
==========

[llandwerlin-old, 2016-01-20 15:53:41]

lionel.g.landwerlin@intel.com changed reviewers:
+ bbudge@chromium.org

[llandwerlin-old, 2016-01-20 15:53:42]

Hi Bill, PTAL,

This is a fix to avoid skipping audio frames due to too much CPU consumption due
to reallocation of the ArrayBuffer containing the data.

Thanks!

[bbudge, 2016-01-20 19:13:09]

lgtm

https://codereview.chromium.org/1569553004/diff/40001/ppapi/examples/audio_en...
File ppapi/examples/audio_encode/audio_encode.html (right):

https://codereview.chromium.org/1569553004/diff/40001/ppapi/examples/audio_en...
ppapi/examples/audio_encode/audio_encode.html:86: // Writes way data.
s/way/wav ?

[llandwerlin-old, 2016-01-22 10:31:01]

On 2016/01/20 19:13:09, bbudge wrote:
> lgtm
> 
>
https://codereview.chromium.org/1569553004/diff/40001/ppapi/examples/audio_en...
> File ppapi/examples/audio_encode/audio_encode.html (right):
> 
>
https://codereview.chromium.org/1569553004/diff/40001/ppapi/examples/audio_en...
> ppapi/examples/audio_encode/audio_encode.html:86: // Writes way data.
> s/way/wav ?

Indeed, thanks!

[llandwerlin-old, 2016-01-22 10:31:06]

The CQ bit was checked by lionel.g.landwerlin@intel.com

[llandwerlin-old, 2016-01-22 10:31:06]

The patchset sent to the CQ was uploaded after l-g-t-m from bbudge@chromium.org
Link to the patchset: https://codereview.chromium.org/1569553004/#ps60001
(title: "comment nit")

[commit-bot: I haz the power, 2016-01-22 10:32:28]

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

[commit-bot: I haz the power, 2016-01-22 10:51:43]

Description was changed from

==========
ppapi: examples: audio_encode: prevent buffer reallocations

Prevent buffer reallocations by using an array of buffers to save encoded
audio data in and just create a buffer to contain all of them when the user
wants to save the data to a file.


BUG=570837
==========

to

==========
ppapi: examples: audio_encode: prevent buffer reallocations

Prevent buffer reallocations by using an array of buffers to save encoded
audio data in and just create a buffer to contain all of them when the user
wants to save the data to a file.


BUG=570837
==========

[commit-bot: I haz the power, 2016-01-22 10:51:44]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-01-22 10:53:21]

Description was changed from

==========
ppapi: examples: audio_encode: prevent buffer reallocations

Prevent buffer reallocations by using an array of buffers to save encoded
audio data in and just create a buffer to contain all of them when the user
wants to save the data to a file.


BUG=570837
==========

to

==========
ppapi: examples: audio_encode: prevent buffer reallocations

Prevent buffer reallocations by using an array of buffers to save encoded
audio data in and just create a buffer to contain all of them when the user
wants to save the data to a file.

BUG=570837

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

[commit-bot: I haz the power, 2016-01-22 10:53:22]

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