Video decode in hardware on ARM platform.

content/common/gpu/media/omx_video_decode_accelerator.h

Index: content/common/gpu/media/omx_video_decode_accelerator.h
===================================================================
--- content/common/gpu/media/omx_video_decode_accelerator.h	(revision 120554)
+++ content/common/gpu/media/omx_video_decode_accelerator.h	(working copy)
@@ -19,8 +19,7 @@
 #include "base/message_loop.h"
 #include "base/shared_memory.h"
 #include "media/video/video_decode_accelerator.h"
-#include "third_party/angle/include/EGL/egl.h"
-#include "third_party/angle/include/EGL/eglext.h"
+#include "content/common/gpu/media/gles2_texture_to_egl_image_translator.h"

[Ami GONE FROM CHROMIUM, 2012/02/18 00:00:25]

Since only a pointer to the translator is held by this class, you can
fwd-declare it and avoid the extra header inclusions.

 #include "third_party/openmax/il/OMX_Component.h"
 #include "third_party/openmax/il/OMX_Core.h"
 #include "third_party/openmax/il/OMX_Video.h"
@@ -148,6 +147,9 @@
   OMX_U32 output_port_;
   int output_buffers_at_component_;
 
+  int width_ ;

[Ami GONE FROM CHROMIUM, 2012/02/18 00:00:25]

prefix both this and the next member to indicate their relevance:
int last_requested_picture_buffer_width_;
int last_requested_picture_buffer_height_;

Although I'd use a gfx::Size instead:
gfx::Size last_requested_picture_buffer_dimensions_;
and also pass a 
const gfx::Size&
to the translator's method instead of independent width/height params.

(using a single variable ensures the order is not confused, and ensures that
both width & height are always set and neither is forgotten)

+  int height_;
+
   // NOTE: someday there may be multiple contexts for a single decoder.  But not
   // today.
   // TODO(fischman,vrk): handle lost contexts?
@@ -179,10 +181,13 @@
   // NOTE: all calls to this object *MUST* be executed in message_loop_.
   Client* client_;
 
+  Gles2TextureToEglImageTranslator* texture2eglImage_translator_;

[Ami GONE FROM CHROMIUM, 2012/02/18 00:00:25]

we don't use '2' as a synonym for 'to', and what we don't use camelCase for
class members.
texture_to_egl_image_translator_

+
   // These two members are only used during Initialization.

[Ami GONE FROM CHROMIUM, 2012/02/18 00:00:25]

s/two //

   // OMX_AVCProfile requested during Initialization.
   uint32 profile_;
   bool component_name_is_nvidia_h264ext_;
+  bool component_name_is_sec_h264ext_;
 
   // Method to handle events
   void EventHandlerCompleteTask(OMX_EVENTTYPE event,