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

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

Issue 596055: Implement GLES video renderer in player_x11 (Closed)
Patch Set: nits fixed Created 10 years, 10 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
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 aa8194b223202d090838363cee24f2a231e6b28b..ffd297694d4f5b56c646d7ae6d115f32d1d15a2c 100644
--- a/media/tools/player_x11/player_x11.cc
+++ b/media/tools/player_x11/player_x11.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2009-2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -21,7 +21,17 @@
#include "media/filters/file_data_source.h"
#include "media/filters/null_audio_renderer.h"
#include "media/filters/omx_video_decoder.h"
+
+#if defined(RENDERER_GL)
+#include "media/tools/player_x11/gl_video_renderer.h"
+#define Renderer GlVideoRenderer
scherkus (not reviewing) 2010/02/12 01:04:20 typedef doesn't work here?
+#elif RENDERER_GLES
+#include "media/tools/player_x11/gles_video_renderer.h"
+#define Renderer GlesVideoRenderer
+#elif RENDERER_X11
#include "media/tools/player_x11/x11_video_renderer.h"
+#define Renderer X11VideoRenderer
+#endif
Display* g_display = NULL;
Window g_window = 0;
@@ -71,7 +81,7 @@ bool InitPipeline(MessageLoop* message_loop,
factories->AddFactory(media::OmxVideoDecoder::CreateFactory());
}
factories->AddFactory(media::FFmpegVideoDecoder::CreateFactory());
- factories->AddFactory(X11VideoRenderer::CreateFactory(g_display, g_window));
+ factories->AddFactory(Renderer::CreateFactory(g_display, g_window));
if (enable_audio) {
factories->AddFactory(media::AudioRendererImpl::CreateFilterFactory());
@@ -146,8 +156,8 @@ int main(int argc, char** argv) {
XNextEvent(g_display, &e);
if (e.type == Expose) {
// Tell the renderer to paint.
- DCHECK(X11VideoRenderer::instance());
- X11VideoRenderer::instance()->Paint();
+ DCHECK(Renderer::instance());
+ Renderer::instance()->Paint();
} else if (e.type == ButtonPress) {
// Stop the playback.
break;

Powered by Google App Engine
This is Rietveld 408576698