Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1206)

Unified Diff: media/tools/player_x11/player_x11.cc

Issue 10854151: Allow transitioning to HAVE_METADATA before pipeline initialization completes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase and address final comments Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/tools/player_wtl/movie.cc ('k') | webkit/media/webmediaplayer_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/tools/player_x11/player_x11.cc
diff --git a/media/tools/player_x11/player_x11.cc b/media/tools/player_x11/player_x11.cc
index 8970b18042cf92f3b9ff4851ecfec08f5383519f..d65389da2823fcde7e6a7a211b76982f07c9b517 100644
--- a/media/tools/player_x11/player_x11.cc
+++ b/media/tools/player_x11/player_x11.cc
@@ -97,6 +97,8 @@ void Paint(MessageLoop* message_loop, const PaintCB& paint_cb) {
g_video_renderer->PutCurrentFrame(video_frame);
}
+static void OnBufferingState(media::Pipeline::BufferingState buffering_state) {}
+
// TODO(vrk): Re-enabled audio. (crbug.com/112159)
bool InitPipeline(const scoped_refptr<base::MessageLoopProxy>& message_loop,
const scoped_refptr<media::DataSource>& data_source,
@@ -140,7 +142,7 @@ bool InitPipeline(const scoped_refptr<base::MessageLoopProxy>& message_loop,
media::PipelineStatusNotification note;
(*pipeline)->Start(
collection.Pass(), media::PipelineStatusCB(), media::PipelineStatusCB(),
- note.Callback());
+ note.Callback(), base::Bind(&OnBufferingState));
// Wait until the pipeline is fully initialized.
note.Wait();
« no previous file with comments | « media/tools/player_wtl/movie.cc ('k') | webkit/media/webmediaplayer_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698