Fix memory leask in media yuv_convert unit tests. Replace pointers init'd by ...

Fix memory leask in media yuv_convert unit tests. Replace pointers init'd by plain new[] with scoped_array.

BUG=20497
(http://crbug.com/20497)
TEST=Valgrind does not report memory leaks any more in media/base/yuv_convert_unittest.cc


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

[fbarchard, 2009-08-28 16:56:02]

LGTM!  Good catch

http://codereview.chromium.org/182010/diff/2001/3001
File media/base/yuv_convert_unittest.cc (right):

http://codereview.chromium.org/182010/diff/2001/3001#newcode142
Line 142: scoped_array<uint8> yuv_bytes(new uint8[size_of_yuv]);
doh!  How quaint.  This was the first unit test I wrote at google.
Good catch
But, I based it directly on one ralphl wrote, so you'll find at least one more
just like this.
My audio_util one may be similar and needs a check, but I think I'm using
scoped_pointers everywhere.

http://codereview.chromium.org/182010/diff/2001/3001#newcode154
Line 154: yuv_bytes.get() + kWidth * kHeight * 5 / 4,  // V
kinda silly that 80 columns forces tiny comments, but this is still readable.