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

media/tools/player_x11/x11_video_renderer.cc

 // 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.
 
 #include "media/tools/player_x11/x11_video_renderer.h"
 
 #include <dlfcn.h>
 #include <X11/Xutil.h>
 #include <X11/extensions/Xrender.h>
 #include <X11/extensions/Xcomposite.h>
@@ skipping 39 matching lines @@
     CHECK(pictformat) << "XRENDER ARGB32 not supported.";
   }
 
   return pictformat;
 }
 
 X11VideoRenderer::X11VideoRenderer(Display* display, Window window)
     : display_(display),
       window_(window),
       image_(NULL),
-      new_frame_(false),
       picture_(0),
-      use_render_(false) {
+      use_render_(false),
+      glx_thread_message_loop_(NULL) {
 }
 
 X11VideoRenderer::~X11VideoRenderer() {
 }
 
 // static
 bool X11VideoRenderer::IsMediaFormatSupported(
     const media::MediaFormat& media_format) {
   int width = 0;
   int height = 0;
@@ skipping 52 matching lines @@
                         width_ * 4);
   DCHECK(image_);
 
   // Save this instance.
   DCHECK(!instance_);
   instance_ = this;
   return true;
 }
 
 void X11VideoRenderer::OnFrameAvailable() {
-  AutoLock auto_lock(lock_);
-  new_frame_ = true;
+  if (glx_thread_message_loop()) {
+    glx_thread_message_loop()->PostTask(FROM_HERE,
+        NewRunnableMethod(this, &X11VideoRenderer::Paint));
+  }
 }
 
 void X11VideoRenderer::Paint() {
-  // Use |new_frame_| to prevent overdraw since Paint() is called more
-  // often than needed. It is OK to lock only this flag and we don't
-  // want to lock the whole function because this method takes a long
-  // time to complete.
-  {
-    AutoLock auto_lock(lock_);
-    if (!new_frame_)
-      return;
-    new_frame_ = false;
-  }
-
   scoped_refptr<media::VideoFrame> video_frame;
   GetCurrentFrame(&video_frame);
 
   if (!image_ ||!video_frame)
     return;
 
   // Convert YUV frame to RGB.
   DCHECK(video_frame->format() == media::VideoFrame::YV12 ||
          video_frame->format() == media::VideoFrame::YV16);
   DCHECK(video_frame->stride(media::VideoFrame::kUPlane) ==
@@ skipping 66 matching lines @@
   // If XRender is not used, simply put the image to the server.
   // This will have a tearing effect but this is OK.
   // TODO(hclam): Upload the image to a pixmap and do XCopyArea()
   // to the window.
   GC gc = XCreateGC(display_, window_, 0, NULL);
   XPutImage(display_, window_, gc, image_,
             0, 0, 0, 0, width_, height_);
   XFlush(display_);
   XFreeGC(display_, gc);
 }