Video decode in hardware on ARM platform.

content/common/gpu/media/gles2_texture_to_egl_image_translator.h

 // Copyright (c) 2011 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 CONTENT_COMMON_GPU_MEDIA_GLES2_TEXTURE_TO_EGL_IMAGE_TRANSLATOR_H_
 #define CONTENT_COMMON_GPU_MEDIA_GLES2_TEXTURE_TO_EGL_IMAGE_TRANSLATOR_H_
 
+#include <utility>

[Ami GONE FROM CHROMIUM, 2012/02/07 17:05:40]

unneeded

+#include <vector>
+#include "base/scoped_ptr.h"

[Ami GONE FROM CHROMIUM, 2012/02/07 17:05:40]

unneeded

+
 #include "base/basictypes.h"
 #include "media/video/picture.h"
 #include "third_party/angle/include/EGL/egl.h"
 #include "third_party/angle/include/EGL/eglext.h"
 #include "third_party/angle/include/GLES2/gl2.h"
 #include "third_party/angle/include/GLES2/gl2ext.h"
 
+#include <X11/Xutil.h>
+#include <X11/extensions/Xrender.h>
+#include <X11/extensions/Xcomposite.h>

[Ami GONE FROM CHROMIUM, 2012/02/07 17:05:40]

None of these includes are necessary, are they?  
(fwd-declare Display if needed)

+
 // Class to wrap egl-opengles related operations.
 // PPAPI will give the textures to OmxVideoDecodeAccelerator.
 // OmxVideoDecodeAccelerator will use this class to convert
 // these texture into EGLImage and back.
 //
 // TODO(fischman): this class is now stateless (it wasn't always).  If it seems
 // like it'll stay stateless, replace with a namespace and free functions, or
 // just in-line them into the only caller, OmxVideoDecodeAccelerator.
 class Gles2TextureToEglImageTranslator {
  public:
   Gles2TextureToEglImageTranslator();
   ~Gles2TextureToEglImageTranslator();
 
   // Translates texture into EGLImage and back.
   EGLImageKHR TranslateToEglImage(EGLDisplay egl_display,
                                   EGLContext egl_context,
                                   uint32 texture);
+  EGLImageKHR CreateEglImage(Display* x_display, EGLDisplay egl_display,

[Ami GONE FROM CHROMIUM, 2012/02/07 17:05:40]

Having a "Create" call per underlying implementation is uncool.
It would be best to fold the implementation details into the .cc file, and
present just a single creation API.

+                             EGLContext egl_context, EGLSurface egl_surface,
+                             uint32 texture, int width, int height);
+
   uint32 TranslateToTexture(EGLImageKHR egl_image);
-  void DestroyEglImage(EGLDisplay egl_display, EGLImageKHR egl_image);
+  void DestroyEglImage(Display* x_display, EGLDisplay egl_display,
+                             EGLImageKHR egl_image);
 
  private:
+ std::vector<Pixmap> pixmap_;
   DISALLOW_COPY_AND_ASSIGN(Gles2TextureToEglImageTranslator);
 };
 
 #endif  // CONTENT_COMMON_GPU_MEDIA_GLES2_TEXTURE_TO_EGL_IMAGE_TRANSLATOR_H_