Attach color space information to hardware decoded NV12 video frames.

media/gpu/ipc/common/media_messages.h

Index: media/gpu/ipc/common/media_messages.h
diff --git a/media/gpu/ipc/common/media_messages.h b/media/gpu/ipc/common/media_messages.h
index a841f1fc89cac9aeb1952865936c65c7b0cf5b3a..c509f761f72e67a32d634cd7c98fef87edc87775 100644
--- a/media/gpu/ipc/common/media_messages.h
+++ b/media/gpu/ipc/common/media_messages.h
@@ -16,6 +16,7 @@
 #include "media/video/jpeg_decode_accelerator.h"
 #include "media/video/video_decode_accelerator.h"
 #include "media/video/video_encode_accelerator.h"
+#include "ui/gfx/ipc/color/gfx_param_traits.h"
 #include "ui/gfx/ipc/gfx_param_traits.h"
 
 #define IPC_MESSAGE_START MediaMsgStart
@@ -119,13 +120,16 @@ IPC_MESSAGE_ROUTED5(AcceleratedVideoDecoderHostMsg_ProvidePictureBuffers,
 IPC_MESSAGE_ROUTED1(AcceleratedVideoDecoderHostMsg_DismissPictureBuffer,
                     int32_t) /* Picture buffer ID */
 
+typedef std::pair<bool, bool> BoolPair;

[Tom Sepez, 2016/10/03 22:44:09]

This needs to be in an include-guarded file, rather than a messages file --
generally the fuzzers break when we violate this rule.  I'd suggest making an
actual struct containing the rect, CS, and the two bools, but it's up to you.

Maybe ui/gfx/ipc/color/gfx_param_traits_macros.h is a good place to do this.

[hubbe, 2016/10/04 20:19:36]

Fixed.

+
 // Decoder reports that a picture is ready.
 IPC_MESSAGE_ROUTED5(AcceleratedVideoDecoderHostMsg_PictureReady,
-                    int32_t,   /* Picture buffer ID */
-                    int32_t,   /* Bitstream buffer ID */
-                    gfx::Rect, /* Visible rectangle */
-                    bool,      /* Buffer is HW overlay capable */
-                    bool)      /* VDA updated picture size */
+                    int32_t,         /* Picture buffer ID */
+                    int32_t,         /* Bitstream buffer ID */
+                    gfx::Rect,       /* Visible rectangle */
+                    gfx::ColorSpace, /* color space */
+                    BoolPair)        /* Buffer is HW overlay capable */
+                                     /* VDA updated picture size */
 
 // Confirm decoder has been flushed.
 IPC_MESSAGE_ROUTED0(AcceleratedVideoDecoderHostMsg_FlushDone)