media: Convert I420 VideoFrame to UYVY GpuMemoryBuffer.

content/renderer/render_thread_impl.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 "content/renderer/render_thread_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <map>
 #include <vector>
@@ skipping 1338 matching lines @@
                 WebGraphicsContext3DCommandBufferImpl::SharedMemoryLimits(),
                 NULL)),
         GPU_VIDEO_ACCELERATOR_CONTEXT);
   }
   if (gpu_va_context_provider_.get()) {
     const bool enable_video_accelerator =
         !cmd_line->HasSwitch(switches::kDisableAcceleratedVideoDecode);
     const bool enable_gpu_memory_buffer_video_frames =
         cmd_line->HasSwitch(switches::kEnableGpuMemoryBufferVideoFrames);
     std::string image_texture_target_string =
         cmd_line->GetSwitchValueASCII(switches::kVideoImageTextureTarget);

[reveman, 2015/08/21 21:53:02]

How can we know what the correct image target is when we haven't decided what
the output format is?

To support multiple output formats, I think we need a vector of targets from the
browser process, just like the compositor.

[Daniele Castagna, 2015/08/21 22:46:04]

Good point. Now we're lucky because on Mac is always texture_rectangle. I'll put
up a CL to fix this.

     unsigned image_texture_target = 0;
     const bool parsed_image_texture_target =
         base::StringToUint(image_texture_target_string, &image_texture_target);
     DCHECK(parsed_image_texture_target);
     CHECK(gpu_channel_.get()) << "Have gpu_va_context_provider but no "
                                  "gpu_channel_. See crbug.com/495185.";
     CHECK(gpu_channel_host.get()) << "Have gpu_va_context_provider but lost "
                                      "gpu_channel_. See crbug.com/495185.";
+    media::VideoPixelFormat video_pixel_format = media::PIXEL_FORMAT_I420;
+#if defined(OS_MACOSX)
+    // TODO(dcastagna): Check that the extension is available.
+    video_pixel_format = media::PIXEL_FORMAT_UYVY;

[reveman, 2015/08/21 21:53:02]

why is this not decided where we check if RG extension is supported?

+#endif
     gpu_factories = RendererGpuVideoAcceleratorFactories::Create(
         gpu_channel_host.get(), media_task_runner, gpu_va_context_provider_,
         enable_gpu_memory_buffer_video_frames, image_texture_target,
-        enable_video_accelerator);
+        video_pixel_format, enable_video_accelerator);
   }
   return gpu_factories;
 }
 
 scoped_ptr<WebGraphicsContext3DCommandBufferImpl>
 RenderThreadImpl::CreateOffscreenContext3d() {
   blink::WebGraphicsContext3D::Attributes attributes(GetOffscreenAttribs());
   bool lose_context_when_out_of_memory = true;
 
   scoped_refptr<GpuChannelHost> gpu_channel_host(EstablishGpuChannelSync(
@@ skipping 578 matching lines @@
 }
 
 void RenderThreadImpl::PendingRenderFrameConnect::OnConnectionError() {
   size_t erased =
       RenderThreadImpl::current()->pending_render_frame_connects_.erase(
           routing_id_);
   DCHECK_EQ(1u, erased);
 }
 
 }  // namespace content