Pepper: Make the sample rate settable in the audio (output) example plugin.

Pepper: Make the sample rate settable in the audio (output) example plugin.

Also do some other cleanup as a drive-by.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112143

[viettrungluu, 2011-11-29 21:22:07]

http://codereview.chromium.org/8676043/diff/1/ppapi/examples/audio/audio.cc
File ppapi/examples/audio/audio.cc (right):

http://codereview.chromium.org/8676043/diff/1/ppapi/examples/audio/audio.cc#n...
ppapi/examples/audio/audio.cc:20: const double frequency_l = 400;
Maybe I should fix these too....

[viettrungluu, 2011-11-30 02:26:31]

+piman, dmichael (since brettw is flaky)

The change is "relatively trivial". Might as well get *some* (but hopefully not
all) OWNERS to review.

http://codereview.chromium.org/8676043/diff/5001/ppapi/examples/audio/audio.cc
File ppapi/examples/audio/audio.cc (right):

http://codereview.chromium.org/8676043/diff/5001/ppapi/examples/audio/audio.c...
ppapi/examples/audio/audio.cc:5: // Needed on Windows to get |M_PI| from math.h
on Windows.
From the Department of Redundancy Department, obviously.

[brettw, 2011-11-30 02:31:00]

I'm not flaky, I was taking several hours to do some actual programming.

LGTM

[viettrungluu, 2011-11-30 03:50:58]

On 2011/11/30 02:31:00, brettw wrote:
> I'm not flaky, I was taking several hours to do some actual programming.
> 
> LGTM

Sorry, I wasn't actually sure if you were working (all of) today. :) Thanks for
the review.

[commit-bot: I haz the power, 2011-11-30 03:51:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/viettrungluu@chromium.org/8676043/7001

[commit-bot: I haz the power, 2011-11-30 06:25:01]

Change committed as 112143