Plumb decoded video pixel format from GPU process to renderer.

media/gpu/ipc/client/gpu_video_decode_accelerator_host.cc

 // Copyright (c) 2012 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/gpu/ipc/client/gpu_video_decode_accelerator_host.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/threading/thread_task_runner_handle.h"
@@ skipping 160 matching lines @@
 }
 
 void GpuVideoDecodeAcceleratorHost::Destroy() {
   DCHECK(CalledOnValidThread());
   if (channel_)
     Send(new AcceleratedVideoDecoderMsg_Destroy(decoder_route_id_));
   client_ = NULL;
   delete this;
 }
 
+VideoPixelFormat GpuVideoDecodeAcceleratorHost::GetOutputFormat() const {
+  return picture_buffer_format_;

[miu, 2016/05/12 20:03:02]

Order-of-operations consideration: Should we DCHECK_NE(PIXEL_FORMAT_UNKNOWN,
picture_buffer_format_) to ensure this isn't called before the format is known?
Or, is it normal for the caller to call GetOutputFormat() and expects an
"UNKNOWN" result before picture buffers are provided?

+}
+
 void GpuVideoDecodeAcceleratorHost::OnWillDeleteImpl() {
   DCHECK(CalledOnValidThread());
   impl_ = NULL;
 
   // The gpu::CommandBufferProxyImpl is going away; error out this VDA.
   OnChannelError();
 }
 
 void GpuVideoDecodeAcceleratorHost::PostNotifyError(Error error) {
   DCHECK(CalledOnValidThread());
@@ skipping 19 matching lines @@
 }
 
 void GpuVideoDecodeAcceleratorHost::OnBitstreamBufferProcessed(
     int32_t bitstream_buffer_id) {
   DCHECK(CalledOnValidThread());
   if (client_)
     client_->NotifyEndOfBitstreamBuffer(bitstream_buffer_id);
 }
 
 void GpuVideoDecodeAcceleratorHost::OnProvidePictureBuffer(
+    VideoPixelFormat format,
     uint32_t num_requested_buffers,
     uint32_t textures_per_buffer,
     const gfx::Size& dimensions,
     uint32_t texture_target) {
   DCHECK(CalledOnValidThread());

[miu, 2016/05/12 20:03:02]

Robustness consideration: Should we DCHECK() that the dimensions and format do
not change? (Could this method be called more than once?)

   picture_buffer_dimensions_ = dimensions;
+  picture_buffer_format_ = format;
 
   const int kMaxVideoPlanes = 4;
   if (textures_per_buffer > kMaxVideoPlanes) {
     PostNotifyError(PLATFORM_FAILURE);
     return;
   }
 
   if (client_) {
     client_->ProvidePictureBuffers(num_requested_buffers, textures_per_buffer,
                                    dimensions, texture_target);
@@ skipping 41 matching lines @@
   weak_this_factory_.InvalidateWeakPtrs();
 
   // Client::NotifyError() may Destroy() |this|, so calling it needs to be the
   // last thing done on this stack!
   VideoDecodeAccelerator::Client* client = NULL;
   std::swap(client, client_);
   client->NotifyError(static_cast<VideoDecodeAccelerator::Error>(error));
 }
 
 }  // namespace media