OmxVideoDecodeAcceleratorTest improvements:

content/common/gpu/omx_video_decode_accelerator.cc

 // 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.
 
 #include "content/common/gpu/omx_video_decode_accelerator.h"
 
 #include "base/stl_util-inl.h"
 #include "base/string_util.h"
 #include "content/common/gpu/gles2_texture_to_egl_image_translator.h"
 #include "content/common/gpu/gpu_channel.h"
@@ skipping 110 matching lines @@
     StopOnError();
     return false;
   }
 
   if (!AllocateInputBuffers()) {
     LOG(ERROR) << "OMX_AllocateBuffer() Input buffer error";
     StopOnError();
     return false;
   }
 
-  // After AllocateInputBuffers ideally this should be AllocateOutputBuffers.
-  // Since in this case app provides the output buffers,
-  // we query this through ProvidePictureBuffers.
-  // This is call to ppapi to provide the output buffers initially.
-  // ProvidePictureBuffers will provide
-  // - SharedMemHandle in case of decoding to system memory.
-  // - Textures in case of decoding to egl-images.
-
-  // Output buffers will be eventually handed to us via
-  // Assign{GLES,Sysmem}Buffers().
+  // In order to allocate output buffers we need to have textures in hand, so we
+  // call ProvidePictureBuffers on the client and complete initialization in
+  // AssignGLESBuffers.
   message_loop_->PostTask(
       FROM_HERE,
       base::Bind(&Client::ProvidePictureBuffers, base::Unretained(client_),
                  static_cast<int32>(kNumPictureBuffers),
                  gfx::Size(width_, height_),
                  PICTUREBUFFER_MEMORYTYPE_GL_TEXTURE));
 
   return true;
 }
 
@@ skipping 761 matching lines @@
   OMX_ERRORTYPE result = OMX_SendCommand(component_handle_,
                                          cmd, port_index, 0);
   if (result != OMX_ErrorNone) {
     LOG(ERROR) << "SendCommand(OMX_CommandPortDisable) failed";
     StopOnError();
     return;
   }
 }
 
 DISABLE_RUNNABLE_METHOD_REFCOUNT(OmxVideoDecodeAccelerator);