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

media/tools/player_x11/gles_video_renderer.h

 // 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.
 
 #ifndef MEDIA_TOOLS_PLAYER_X11_GL_VIDEO_RENDERER_H_
 #define MEDIA_TOOLS_PLAYER_X11_GL_VIDEO_RENDERER_H_
 
 #include <EGL/egl.h>
 #include <EGL/eglext.h>
 #include <GLES2/gl2.h>
@@ skipping 17 matching lines @@
 
   // This method is called to paint the current video frame to the assigned
   // window.
   void Paint();
 
   // media::FilterFactoryImpl2 Implementation.
   static bool IsMediaFormatSupported(const media::MediaFormat& media_format);
 
   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);
   virtual void OnStop();
   virtual void OnFrameAvailable();
 
  private:
   // Only allow to be deleted by reference counting.
   friend class scoped_refptr<GlesVideoRenderer>;
   virtual ~GlesVideoRenderer();
 
   bool InitializeGles();
   void CreateShader(GLuint program, GLenum type,
                     const char* vs_source, int vs_size);
   void LinkProgram(GLuint program);
   void CreateTextureAndProgramEgl();
   void CreateTextureAndProgramYuv2Rgb();
 
   PFNEGLCREATEIMAGEKHRPROC egl_create_image_khr_;
   PFNEGLDESTROYIMAGEKHRPROC egl_destroy_image_khr_;
   PFNGLEGLIMAGETARGETTEXTURE2DOESPROC gl_eglimage_target_texture2d_oes_;
 
   int width_;
   int height_;
   bool uses_egl_image_;
 
   Display* display_;
   Window window_;
 
-  // Protects |new_frame_|.
-  Lock lock_;
-  bool new_frame_;
-
   // EGL context.
   EGLDisplay egl_display_;
   EGLSurface egl_surface_;
   EGLContext egl_context_;
 
   // textures for EGL image
   typedef std::pair<scoped_refptr<media::VideoFrame>, GLuint> EglFrame;
   std::vector<EglFrame> egl_frames_;
 
   // 3 textures, one for each plane.
   GLuint textures_[3];
 
+  MessageLoop* glx_thread_message_loop_;
   static GlesVideoRenderer* instance_;
 
   DISALLOW_COPY_AND_ASSIGN(GlesVideoRenderer);
 };
 
 #endif  // MEDIA_TOOLS_PLAYER_X11_GLES_VIDEO_RENDERER_H_