player_x11 : change X/GL thread to message loop for injecting task

media/tools/player_x11/gles_video_renderer.h

Index: media/tools/player_x11/gles_video_renderer.h
diff --git a/media/tools/player_x11/gles_video_renderer.h b/media/tools/player_x11/gles_video_renderer.h
index d93fa2e6073bc60cc8fc4e3caffd1867d1023c21..2aaa19ccebf45ea66a4c2df61d8144ba909bf23f 100644
--- a/media/tools/player_x11/gles_video_renderer.h
+++ b/media/tools/player_x11/gles_video_renderer.h
@@ -35,6 +35,14 @@ class GlesVideoRenderer : public media::VideoRendererBase {
 
   static GlesVideoRenderer* instance() { return instance_; }
 
+  void set_glx_thread_message_loop(MessageLoop* message_loop) {
+    glx_thread_message_loop_ = message_loop;
+  }

[wjia(left Chromium), 2010/06/09 01:08:06]

Is it possible to use the message_loop in MediaFilter?

+
+  MessageLoop* glx_thread_message_loop() {

[jiesun, 2010/06/09 01:18:36]

MediaFilter::message_loop_ mostly mean its own message loop on it own thread,
not someone else's message loop.
If in the future, we need a message loop based VideoRenderBase, then this
overload will be troublesome and confusing. thanks

+    return glx_thread_message_loop_;
+  }
+
  protected:
   // VideoRendererBase implementation.
   virtual bool OnInitialize(media::VideoDecoder* decoder);
@@ -64,10 +72,6 @@ class GlesVideoRenderer : public media::VideoRendererBase {
   Display* display_;
   Window window_;
 
-  // Protects |new_frame_|.
-  Lock lock_;
-  bool new_frame_;
-
   // EGL context.
   EGLDisplay egl_display_;
   EGLSurface egl_surface_;
@@ -80,6 +84,7 @@ class GlesVideoRenderer : public media::VideoRendererBase {
   // 3 textures, one for each plane.
   GLuint textures_[3];
 
+  MessageLoop* glx_thread_message_loop_;
   static GlesVideoRenderer* instance_;
 
   DISALLOW_COPY_AND_ASSIGN(GlesVideoRenderer);