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

media/video/video_decode_accelerator.h

 // 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.
 
 #ifndef MEDIA_VIDEO_VIDEO_DECODE_ACCELERATOR_H_
 #define MEDIA_VIDEO_VIDEO_DECODE_ACCELERATOR_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 16 matching lines @@
   // |max_resolution| and |min_resolution| are inclusive.
   struct MEDIA_EXPORT SupportedProfile {
     SupportedProfile();
     ~SupportedProfile();
     VideoCodecProfile profile;
     gfx::Size max_resolution;
     gfx::Size min_resolution;
   };
   using SupportedProfiles = std::vector<SupportedProfile>;
 
+  struct MEDIA_EXPORT Capabilities {
+    Capabilities();
+    ~Capabilities();
+    // Flags that can be associated with a VDA.
+    enum Flags {
+      kNoFlags = 0,

[sandersd (OOO until July 31), 2015/12/04 19:10:24]

And now here we've ended up with two different enum styles in the same file. I
don't even know what I want anymore :-(

(Relatedly: there were at least two chromium-dev threads about bitfield enums,
and they didn't come to any conclusions other than that contexpr is still
banned.)

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

true.  changed to NO_FLAGS to be consistent. changing everything else to
kIllegalState is just too much for this CL.

+
+      // If set, then the VDA makes no guarantees that it won't stall before
+      // sending all the picture buffers to the client via PictureReady.
+      kCanStallAnytime = 1 << 0,
+    };
+
+    SupportedProfiles supported_profiles;
+    Flags flags;
+  };
+
   // Enumeration of potential errors generated by the API.
   // Note: Keep these in sync with PP_VideoDecodeError_Dev. Also do not
   // rearrange, reuse or remove values as they are used for gathering UMA
   // statistics.
   enum Error {
     // An operation was attempted during an incompatible decoder state.
     ILLEGAL_STATE = 1,
     // Invalid argument was passed to an API method.
     INVALID_ARGUMENT,
     // Encoded input is unreadable.
@@ skipping 149 matching lines @@
 // Specialize std::default_delete so that scoped_ptr<VideoDecodeAccelerator>
 // uses "Destroy()" instead of trying to use the destructor.
 template <>
 struct MEDIA_EXPORT default_delete<media::VideoDecodeAccelerator> {
   void operator()(media::VideoDecodeAccelerator* vda) const;
 };
 
 }  // namespace std
 
 #endif  // MEDIA_VIDEO_VIDEO_DECODE_ACCELERATOR_H_