Don't use an unretained read callback when calling AudioDecoder::Read().

Don't use an unretained read callback when calling AudioDecoder::Read().

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

[scherkus (not reviewing), 2012-08-17 23:01:40]

PTAL -- candidate fix

potential worrisome bit:  will we leak memory due to a circular refcount between
renderer <-> decoder?

AFAIK this *shouldn't* be an issue since during teardown we require the pending
read to complete.

[scherkus (not reviewing), 2012-08-17 23:27:37]

ran through the gauntlet of valgrind tests and no leaks reported

[Ami GONE FROM CHROMIUM, 2012-08-17 23:46:03]

lgtm

http://codereview.chromium.org/10857066/diff/2001/media/filters/audio_rendere...
File media/filters/audio_renderer_impl.cc (left):

http://codereview.chromium.org/10857066/diff/2001/media/filters/audio_rendere...
media/filters/audio_renderer_impl.cc:31: base::Unretained(this))) {
Unretained(this) for *speed*.

[scherkus (not reviewing), 2012-08-17 23:51:54]

http://codereview.chromium.org/10857066/diff/2001/media/filters/audio_rendere...
File media/filters/audio_renderer_impl.cc (left):

http://codereview.chromium.org/10857066/diff/2001/media/filters/audio_rendere...
media/filters/audio_renderer_impl.cc:31: base::Unretained(this))) {
On 2012/08/17 23:46:03, Ami Fischman wrote:
> Unretained(this) for *speed*.

I don't know why I did this. Must have been smoking something.