Plumb allow_overlay flag for video path into cc

media/video/picture.h

 // Copyright (c) 2011 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_PICTURE_H_
 #define MEDIA_VIDEO_PICTURE_H_
 
 #include "base/basictypes.h"
 #include "gpu/command_buffer/common/mailbox.h"
 #include "media/base/media_export.h"
@@ skipping 39 matching lines @@
   uint32 texture_id_;
   gpu::Mailbox texture_mailbox_;
 };
 
 // A decoded picture frame.
 // This is the media-namespace equivalent of PP_Picture_Dev.
 class MEDIA_EXPORT Picture {
  public:
   Picture(int32 picture_buffer_id,
           int32 bitstream_buffer_id,
-          const gfx::Rect& visible_rect);
+          const gfx::Rect& visible_rect,
+          bool allow_overlay);

[Pawel Osciak, 2015/01/21 13:09:03]

Could we get this property from PictureBuffer? Do we need it on both Picture and
PictureBuffer?

[achaulk, 2015/01/21 17:12:03]

It doesn't seem to be required on the PictureBuffer, at least for the current
use

 
   // Returns the id of the picture buffer where this picture is contained.
   int32 picture_buffer_id() const {
     return picture_buffer_id_;
   }
 
   // Returns the id of the bitstream buffer from which this frame was decoded.
   int32 bitstream_buffer_id() const {
     return bitstream_buffer_id_;
   }
 
   void set_bitstream_buffer_id(int32 bitstream_buffer_id) {
     bitstream_buffer_id_ = bitstream_buffer_id;
   }
 
   // Returns the visible rectangle of the picture. Its size may be smaller
   // than the size of the PictureBuffer, as it is the only visible part of the
   // Picture contained in the PictureBuffer.
   gfx::Rect visible_rect() const { return visible_rect_; }
 
+  bool allow_overlay() const { return allow_overlay_; }
+
  private:
   int32 picture_buffer_id_;
   int32 bitstream_buffer_id_;
   gfx::Rect visible_rect_;
+  bool allow_overlay_;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_VIDEO_PICTURE_H_