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

media/renderers/gpu_video_accelerator_factories.h

Index: media/renderers/gpu_video_accelerator_factories.h
diff --git a/media/renderers/gpu_video_accelerator_factories.h b/media/renderers/gpu_video_accelerator_factories.h
index 329f4b4e1a69882bf881a7e352cf4452623d04fc..c9ac36fbbcf6e28682d645be1e156269b8f93643 100644
--- a/media/renderers/gpu_video_accelerator_factories.h
+++ b/media/renderers/gpu_video_accelerator_factories.h
@@ -97,9 +97,10 @@ class MEDIA_EXPORT GpuVideoAcceleratorFactories {
   // Returns the task runner the video accelerator runs on.
   virtual scoped_refptr<base::SingleThreadTaskRunner> GetTaskRunner() = 0;
 
-  // Returns the supported codec profiles of video decode accelerator.
-  virtual VideoDecodeAccelerator::SupportedProfiles
-      GetVideoDecodeAcceleratorSupportedProfiles() = 0;
+  // Return the capabilities of video decode accelerator, which includes the
+  // supported codec profiles.
+  virtual VideoDecodeAccelerator::Capabilities
+  GetVideoDecodeAcceleratorCapabilities() = 0;

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

Indent?

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

cl format will simply put it back.

[Pawel Osciak, 2015/12/09 01:31:48]

It's strange it doesn't put back l.107. This may be a bug in git cl format
(either way).

[dcheng, 2015/12/09 01:41:50]

I don't think this is a bug. This is one of the points that the style guide
doesn't explicitly make a ruling on, so clang-format just picked one way to be
consistent.

You could make a good argument that this is how templates are formatted, e.g.:
template <typename T>
class scoped_ptr {
  // ...
};

 
   // Returns the supported codec profiles of video encode accelerator.
   virtual VideoEncodeAccelerator::SupportedProfiles