Implementing preload=metadata for video

media/filters/ffmpeg_demuxer.cc

Index: media/filters/ffmpeg_demuxer.cc
diff --git a/media/filters/ffmpeg_demuxer.cc b/media/filters/ffmpeg_demuxer.cc
index d0d4c349df7ea598b1a74fe9f1e57d4241634388..a109e713597d5b3a11cc81489bdf0332b95401b6 100644
--- a/media/filters/ffmpeg_demuxer.cc
+++ b/media/filters/ffmpeg_demuxer.cc
@@ -16,6 +16,7 @@
 #include "media/filters/bitstream_converter.h"
 #include "media/filters/ffmpeg_demuxer.h"
 #include "media/filters/ffmpeg_glue.h"
+#include "webkit/glue/media/defer_strategy.h"
 #include "media/filters/ffmpeg_h264_bitstream_converter.h"
 
 namespace media {
@@ -310,6 +311,16 @@ void FFmpegDemuxer::Seek(base::TimeDelta time, FilterCallback* callback) {
       NewRunnableMethod(this, &FFmpegDemuxer::SeekTask, time, callback));
 }
 
+void FFmpegDemuxer::SetPlaybackRate(float playback_rate) {

[acolwell GONE FROM CHROMIUM, 2011/03/25 04:35:28]

This is necessary because we accidentally broke the existing functionality when
DataSources became owned by the demuxer instead of the CompositeFilter? Just
checking.

If so could you add a unit test to make sure we capture this behavior.

[vrk (LEFT CHROMIUM), 2011/03/25 21:33:32]

Sure, will do!

+  DCHECK(data_source_.get());
+  data_source_->SetPlaybackRate(playback_rate);
+}
+
+void FFmpegDemuxer::SetPreload(media::Preload preload) {

[acolwell GONE FROM CHROMIUM, 2011/03/25 04:35:28]

shouldn't need media::

[vrk (LEFT CHROMIUM), 2011/03/25 21:33:32]

Done.

+  DCHECK(data_source_.get());
+  data_source_->SetPreload(preload);
+}
+
 void FFmpegDemuxer::OnAudioRendererDisabled() {
   message_loop_->PostTask(FROM_HERE,
       NewRunnableMethod(this, &FFmpegDemuxer::DisableAudioStreamTask));