Change DemuxerStream::ReadCallback to use ref-counted buffers.

Change DemuxerStream::ReadCallback to use ref-counted buffers.

Previously callbackees had to remember to store the buffer in a scoped_refptr<>.

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

[Ami GONE FROM CHROMIUM, 2011-11-01 19:30:57]

LGTM

*somebody* is having too much fun!

http://codereview.chromium.org/8351045/diff/1001/media/filters/ffmpeg_video_d...
File media/filters/ffmpeg_video_decoder.cc (right):

http://codereview.chromium.org/8351045/diff/1001/media/filters/ffmpeg_video_d...
media/filters/ffmpeg_video_decoder.cc:183: void
FFmpegVideoDecoder::OnReadCompleteTask(
Elsewhere (i.e. FFmpegVideoDecoder::Seek) this file uses the idiom of
dispatching-to-self if on the wrong loop; wanna drop this *Task() method to do
the same?

[scherkus (not reviewing), 2011-11-01 22:10:25]

Discussed offline -- I can't make the OnReadComplete change because we end up
creating a chain of synchronous calls (i.e., what is supposed to be an async
operation became synchronous and reentrant)

Will leave as is but update TODO -- this should get resolved in
http://codereview.chromium.org/8417019/ anyway