V4L2VDA: use YV12 as output format if processor supports it.

media/gpu/v4l2_video_decode_accelerator.cc

 // Copyright 2014 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/v4l2_video_decode_accelerator.h"
 
 #include <dlfcn.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <linux/videodev2.h>
@@ skipping 2020 matching lines @@
                   fmtdesc.pixelformat) != processor_input_formats.end()) {
       DVLOGF(1) << "Image processor input format=" << fmtdesc.pixelformat;
       return fmtdesc.pixelformat;
     }
     ++fmtdesc.index;
   }
   return 0;
 }
 
 uint32_t V4L2VideoDecodeAccelerator::FindImageProcessorOutputFormat() {
+  // Prefer YVU420 and NV12 because ArcGpuVideoDecodeAccelerator only support

[Pawel Osciak, 2016/08/23 05:22:00]

s/support/supports/

[wuchengli, 2016/08/23 08:52:32]

Done.

+  // single physical plane. Prefer YVU420 over NV12 because chrome rendering
+  // supports YV12 only.
+  static const uint32_t kPreferredFormats[] = {V4L2_PIX_FMT_YVU420,
+                                               V4L2_PIX_FMT_NV12};
+  auto preferred_formats_first = [](uint32_t a, uint32_t b) -> bool {
+    auto iter_a = std::find(std::begin(kPreferredFormats),
+                            std::end(kPreferredFormats), a);
+    auto iter_b = std::find(std::begin(kPreferredFormats),
+                            std::end(kPreferredFormats), b);
+    return iter_a < iter_b;
+  };
+
   V4L2ImageProcessor image_processor(image_processor_device_);
   std::vector<uint32_t> processor_output_formats =
       image_processor.GetSupportedOutputFormats();
+
+  // Move the preferred format to the front.

[Pawel Osciak, 2016/08/23 05:22:00]

s/format/formats/
/./in the same order as in kPreferredFormats./

[wuchengli, 2016/08/23 08:52:33]

Done.

+  std::sort(processor_output_formats.begin(), processor_output_formats.end(),
+            preferred_formats_first);
+
   for (uint32_t processor_output_format : processor_output_formats) {
     if (device_->CanCreateEGLImageFrom(processor_output_format)) {
       DVLOGF(1) << "Image processor output format=" << processor_output_format;
       return processor_output_format;
     }
   }
 
   return 0;
 }
 
@@ skipping 217 matching lines @@
       Enqueue();
   }
 }
 
 void V4L2VideoDecodeAccelerator::ImageProcessorError() {
   LOGF(ERROR) << "Image processor error";
   NOTIFY_ERROR(PLATFORM_FAILURE);
 }
 
 }  // namespace media