Don't require VDAs to return all PictureBuffers at once.

content/common/gpu/media/gpu_video_accelerator_util.cc

 // Copyright 2015 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/media/gpu_video_accelerator_util.h"
 
 namespace content {
 
 // Make sure the enum values of media::VideoCodecProfile and
 // gpu::VideoCodecProfile match.
@@ skipping 13 matching lines @@
 STATIC_ASSERT_ENUM_MATCH(H264PROFILE_HIGH444PREDICTIVEPROFILE);
 STATIC_ASSERT_ENUM_MATCH(H264PROFILE_SCALABLEBASELINE);
 STATIC_ASSERT_ENUM_MATCH(H264PROFILE_SCALABLEHIGH);
 STATIC_ASSERT_ENUM_MATCH(H264PROFILE_STEREOHIGH);
 STATIC_ASSERT_ENUM_MATCH(H264PROFILE_MULTIVIEWHIGH);
 STATIC_ASSERT_ENUM_MATCH(VP8PROFILE_ANY);
 STATIC_ASSERT_ENUM_MATCH(VP9PROFILE_ANY);
 STATIC_ASSERT_ENUM_MATCH(VIDEO_CODEC_PROFILE_MAX);
 
 // static
+media::VideoDecodeAccelerator::Capabilities
+GpuVideoAcceleratorUtil::ConvertGpuToMediaDecodeCapabilities(
+    const gpu::VideoDecodeAcceleratorCapabilities& gpu_capabilities) {
+  media::VideoDecodeAccelerator::Capabilities capabilities;
+  capabilities.supported_profiles =
+      ConvertGpuToMediaDecodeProfiles(gpu_capabilities.supported_profiles);
+  capabilities.flags =
+      static_cast<media::VideoDecodeAccelerator::Capabilities::Flags>(

[Pawel Osciak, 2015/12/05 00:18:55]

Is the cast needed? Both are uint32s...

[liberato (no reviews please), 2015/12/07 19:04:39]

yes: error: invalid conversion from 'uint32 {aka unsigned int}' to
'media::VideoDecodeAccelerator::Capabilities::Flags' [-fpermissive]

plus, it makes the intention more explicit.

+          gpu_capabilities.flags);
+  return capabilities;
+}
+
+// static
 media::VideoDecodeAccelerator::SupportedProfiles
 GpuVideoAcceleratorUtil::ConvertGpuToMediaDecodeProfiles(const
     gpu::VideoDecodeAcceleratorSupportedProfiles& gpu_profiles) {
   media::VideoDecodeAccelerator::SupportedProfiles profiles;
   for (const auto& gpu_profile : gpu_profiles) {
     media::VideoDecodeAccelerator::SupportedProfile profile;
     profile.profile =
         static_cast<media::VideoCodecProfile>(gpu_profile.profile);
     profile.max_resolution = gpu_profile.max_resolution;
     profile.min_resolution = gpu_profile.min_resolution;
     profiles.push_back(profile);
   }
   return profiles;
 }
 
 // static
+gpu::VideoDecodeAcceleratorCapabilities
+GpuVideoAcceleratorUtil::ConvertMediaToGpuDecodeCapabilities(
+    const media::VideoDecodeAccelerator::Capabilities& media_capabilities) {
+  gpu::VideoDecodeAcceleratorCapabilities capabilities;
+  capabilities.supported_profiles =
+      ConvertMediaToGpuDecodeProfiles(media_capabilities.supported_profiles);
+  capabilities.flags = static_cast<uint32>(media_capabilities.flags);
+  return capabilities;
+}
+
+// static
 gpu::VideoDecodeAcceleratorSupportedProfiles
 GpuVideoAcceleratorUtil::ConvertMediaToGpuDecodeProfiles(const
     media::VideoDecodeAccelerator::SupportedProfiles& media_profiles) {
   gpu::VideoDecodeAcceleratorSupportedProfiles profiles;
   for (const auto& media_profile : media_profiles) {
     gpu::VideoDecodeAcceleratorSupportedProfile profile;
     profile.profile =
         static_cast<gpu::VideoCodecProfile>(media_profile.profile);
     profile.max_resolution = media_profile.max_resolution;
     profile.min_resolution = media_profile.min_resolution;
@@ skipping 64 matching lines @@
         duplicate = true;
         break;
       }
     }
     if (!duplicate)
       media_profiles->push_back(profile);
   }
 }
 
 }  // namespace content