[Ozone] Enables overlay render format setting path and by default use UYVY

content/common/gpu/media/vaapi_video_encode_accelerator.cc

Index: content/common/gpu/media/vaapi_video_encode_accelerator.cc
diff --git a/content/common/gpu/media/vaapi_video_encode_accelerator.cc b/content/common/gpu/media/vaapi_video_encode_accelerator.cc
index 1f9388e33abb400d88fbd28fbfdc27f228d1bb8a..9872cbb7e100e1637d80ff126768fa9dd7526b04 100644
--- a/content/common/gpu/media/vaapi_video_encode_accelerator.cc
+++ b/content/common/gpu/media/vaapi_video_encode_accelerator.cc
@@ -582,12 +582,14 @@ bool VaapiVideoEncodeAccelerator::PrepareNextJob() {
     return false;
   }
 
-  current_encode_job_->input_surface = new VASurface(
-      available_va_surface_ids_.back(), coded_size_, va_surface_release_cb_);
+  current_encode_job_->input_surface =
+      new VASurface(available_va_surface_ids_.back(), coded_size_,
+                    VA_RT_FORMAT_YUV420, va_surface_release_cb_);

[Pawel Osciak, 2015/11/05 10:23:51]

We shouldn't be hardcoding the format in multiple places like this, as I
mentioned previously. It's easy to not change this later, or have some other
mismatch somewhere.

We'd have to use a separate struct instead of VASurfaceID vector in VaapiWrapper
for this though. But since the surface set is tied to current VaapiWrapper
state, I'm ok adding a surface_format_ member to VaapiWrapper and an ability to
query for it instead.

So VaapiWrapper::CreateSurfaces() would create surfaces and store the format of
the created surfaces in VaapiWrapper. Then we could query the wrapper for the
current format here and in other places.

[william.xie1, 2015/11/06 06:56:14]

Done.

   available_va_surface_ids_.pop_back();
 
-  current_encode_job_->recon_surface = new VASurface(
-      available_va_surface_ids_.back(), coded_size_, va_surface_release_cb_);
+  current_encode_job_->recon_surface =
+      new VASurface(available_va_surface_ids_.back(), coded_size_,
+                    VA_RT_FORMAT_YUV420, va_surface_release_cb_);
   available_va_surface_ids_.pop_back();
 
   // Reference surfaces are needed until the job is done, but they get