code fix for video frame

media/tools/player_x11/gles_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/gles_video_renderer.h"
 
 #include <dlfcn.h>
 #include <X11/Xutil.h>
 #include <X11/extensions/Xrender.h>
 #include <X11/extensions/Xcomposite.h>
 
 #include "media/base/buffers.h"
 #include "media/base/pipeline.h"
 #include "media/base/filter_host.h"
+#include "media/base/video_frame.h"
 #include "media/base/yuv_convert.h"
 
 GlesVideoRenderer* GlesVideoRenderer::instance_ = NULL;
 
 GlesVideoRenderer::GlesVideoRenderer(Display* display, Window window)
     : display_(display),
       window_(window),
       new_frame_(false),
       egl_display_(NULL),
       egl_surface_(NULL),
@@ skipping 117 matching lines @@
   }
   initialized = true;
 
   scoped_refptr<media::VideoFrame> video_frame;
   GetCurrentFrame(&video_frame);
 
   if (!video_frame)
     return;
 
   // Convert YUV frame to RGB.
-  media::VideoSurface frame_in;
-  if (video_frame->Lock(&frame_in)) {
-    DCHECK(frame_in.format == media::VideoSurface::YV12 ||
-           frame_in.format == media::VideoSurface::YV16);
-    DCHECK(frame_in.strides[media::VideoSurface::kUPlane] ==
-           frame_in.strides[media::VideoSurface::kVPlane]);
-    DCHECK(frame_in.planes == media::VideoSurface::kNumYUVPlanes);
+  DCHECK(video_frame->format() == media::VideoFrame::YV12 ||
+         video_frame->format() == media::VideoFrame::YV16);
+  DCHECK(video_frame->stride(media::VideoFrame::kUPlane) ==
+         video_frame->stride(media::VideoFrame::kVPlane));
+  DCHECK(video_frame->planes() == media::VideoFrame::kNumYUVPlanes);
 
-    for (unsigned int i = 0; i < media::VideoSurface::kNumYUVPlanes; ++i) {
-      unsigned int width = (i == media::VideoSurface::kYPlane) ?
-          frame_in.width : frame_in.width / 2;
-      unsigned int height = (i == media::VideoSurface::kYPlane ||
-                             frame_in.format == media::VideoSurface::YV16) ?
-          frame_in.height : frame_in.height / 2;
-      glActiveTexture(GL_TEXTURE0 + i);
-      // GLES2 supports a fixed set of unpack alignments that should match most
-      // of the time what ffmpeg outputs.
-      // TODO(piman): check if it is more efficient to prefer higher
-      // alignments.
-      unsigned int stride = frame_in.strides[i];
-      uint8* data = frame_in.data[i];
-      if (stride == width) {
-        glPixelStorei(GL_UNPACK_ALIGNMENT, 1);
-      } else if (stride == ((width + 1) & ~1)) {
-        glPixelStorei(GL_UNPACK_ALIGNMENT, 2);
-      } else if (stride == ((width + 3) & ~3)) {
-        glPixelStorei(GL_UNPACK_ALIGNMENT, 4);
-      } else if (stride == ((width + 7) & ~7)) {
-        glPixelStorei(GL_UNPACK_ALIGNMENT, 8);
-      } else {
-        // Otherwise do it line-by-line.
-        glTexImage2D(GL_TEXTURE_2D, 0, GL_LUMINANCE, width, height, 0,
-                     GL_LUMINANCE, GL_UNSIGNED_BYTE, NULL);
-        for (unsigned int y = 0; y < height; ++y) {
-          glTexSubImage2D(GL_TEXTURE_2D, 0, 0, y, width, 1,
-                          GL_LUMINANCE, GL_UNSIGNED_BYTE, data);
-          data += stride;
-        }
-        continue;
+  for (unsigned int i = 0; i < media::VideoFrame::kNumYUVPlanes; ++i) {
+    unsigned int width = (i == media::VideoFrame::kYPlane) ?
+        video_frame->width() : video_frame->width() / 2;
+    unsigned int height = (i == media::VideoFrame::kYPlane ||
+                          video_frame->format() == media::VideoFrame::YV16) ?
+                          video_frame->height() : video_frame->height() / 2;
+    glActiveTexture(GL_TEXTURE0 + i);
+    // GLES2 supports a fixed set of unpack alignments that should match most
+    // of the time what ffmpeg outputs.
+    // TODO(piman): check if it is more efficient to prefer higher
+    // alignments.
+    unsigned int stride = video_frame->stride(i);
+    uint8* data = video_frame->data(i);
+    if (stride == width) {
+      glPixelStorei(GL_UNPACK_ALIGNMENT, 1);
+    } else if (stride == ((width + 1) & ~1)) {
+      glPixelStorei(GL_UNPACK_ALIGNMENT, 2);
+    } else if (stride == ((width + 3) & ~3)) {
+      glPixelStorei(GL_UNPACK_ALIGNMENT, 4);
+    } else if (stride == ((width + 7) & ~7)) {
+      glPixelStorei(GL_UNPACK_ALIGNMENT, 8);
+    } else {
+      // Otherwise do it line-by-line.
+      glTexImage2D(GL_TEXTURE_2D, 0, GL_LUMINANCE, width, height, 0,
+                   GL_LUMINANCE, GL_UNSIGNED_BYTE, NULL);
+      for (unsigned int y = 0; y < height; ++y) {
+        glTexSubImage2D(GL_TEXTURE_2D, 0, 0, y, width, 1,
+                        GL_LUMINANCE, GL_UNSIGNED_BYTE, data);
+        data += stride;
       }
-      glTexImage2D(GL_TEXTURE_2D, 0, GL_LUMINANCE, width, height, 0,
-                   GL_LUMINANCE, GL_UNSIGNED_BYTE, data);
+      continue;
     }
-    video_frame->Unlock();
-  } else {
-    NOTREACHED();
+
+    glTexImage2D(GL_TEXTURE_2D, 0, GL_LUMINANCE, width, height, 0,
+                 GL_LUMINANCE, GL_UNSIGNED_BYTE, data);
   }
 
   glDrawArrays(GL_TRIANGLE_STRIP, 0, 4);
   eglSwapBuffers(egl_display_, egl_surface_);
 }
 
 bool GlesVideoRenderer::InitializeGles() {
   // Resize the window to fit that of the video.
   XResizeWindow(display_, window_, width_, height_);
 
@@ skipping 74 matching lines @@
   }
 
   EGLint width;
   EGLint height;
   eglQuerySurface(egl_display_, egl_surface_, EGL_WIDTH, &width);
   eglQuerySurface(egl_display_, egl_surface_, EGL_HEIGHT, &height);
   glViewport(0, 0, width_, height_);
 
   // Create 3 textures, one for each plane, and bind them to different
   // texture units.
-  glGenTextures(media::VideoSurface::kNumYUVPlanes, textures_);
+  glGenTextures(media::VideoFrame::kNumYUVPlanes, textures_);
 
   glActiveTexture(GL_TEXTURE0);
   glBindTexture(GL_TEXTURE_2D, textures_[0]);
   glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
   glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
   glEnable(GL_TEXTURE_2D);
 
   glActiveTexture(GL_TEXTURE1);
   glBindTexture(GL_TEXTURE_2D, textures_[1]);
   glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
@@ skipping 73 matching lines @@
                         kTextureCoords);
 
   // We are getting called on a thread. Release the context so that it can be
   // made current on the main thread.
   // TODO(hclam): Fix this if neccessary. Currently the following call fails
   // for some drivers.
   // eglMakeCurrent(egl_display_, EGL_NO_SURFACE,
   //                EGL_NO_SURFACE, EGL_NO_CONTEXT);
   return true;
 }