Initialize the shared memory to appease Valgrind.

Initialize the shared memory to appease Valgrind.

BUG=138515
TEST=Under valgrind, run: content_unittests --gtest_filter=AudioDeviceTest.*

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=147848

[henrika (OOO until Aug 14), 2012-07-23 11:17:06]

LGTM w/ nits.

But linux_valgrind is read...

https://chromiumcodereview.appspot.com/10806061/diff/1003/content/renderer/me...
File content/renderer/media/audio_device_unittest.cc (right):

https://chromiumcodereview.appspot.com/10806061/diff/1003/content/renderer/me...
content/renderer/media/audio_device_unittest.cc:116: int number_of_frames,
Odd indentation. int is not aligned with const.

https://chromiumcodereview.appspot.com/10806061/diff/1003/content/renderer/me...
content/renderer/media/audio_device_unittest.cc:246: WithArgs<0, 1,
2>(Invoke(&FillData)),
Mind adding a comment?

https://chromiumcodereview.appspot.com/10806061/diff/1003/content/renderer/me...
content/renderer/media/audio_device_unittest.cc:248: Return(1)));
Why 1 here?

[tommi (sloooow) - chröme, 2012-07-23 12:23:09]

A few try runs were red due to patch problems (the revision that updated the
suppressions was newer than lkgr).

I've changed things around a bit now as discussed offline.
AudioDevice::AudioThreadCallback::Process() was actually passing the incorrect
number of frames to InterleaveFloatToInt() and not the number of frames as
returned from the Render() callback.

PTAL.

https://chromiumcodereview.appspot.com/10806061/diff/1003/content/renderer/me...
File content/renderer/media/audio_device_unittest.cc (right):

https://chromiumcodereview.appspot.com/10806061/diff/1003/content/renderer/me...
content/renderer/media/audio_device_unittest.cc:116: int number_of_frames,
On 2012/07/23 11:17:07, henrika wrote:
> Odd indentation. int is not aligned with const.

Done.

https://chromiumcodereview.appspot.com/10806061/diff/1003/content/renderer/me...
content/renderer/media/audio_device_unittest.cc:246: WithArgs<0, 1,
2>(Invoke(&FillData)),
On 2012/07/23 11:17:07, henrika wrote:
> Mind adding a comment?

Done.

[henrika (OOO until Aug 14), 2012-07-23 13:02:15]

LGTM.

[commit-bot: I haz the power, 2012-07-23 13:11:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tommi@chromium.org/10806061/8005