Decouple Media Service

content/common/gpu/media/media_messages.h

Index: content/common/gpu/media/media_messages.h
diff --git a/content/common/gpu/media_messages.h b/content/common/gpu/media/media_messages.h
similarity index 89%
rename from content/common/gpu/media_messages.h
rename to content/common/gpu/media/media_messages.h
index de0affbb3ee64ee39b98b91730a241f831add9e6..fc3b01851b1eb062d54b1585ac74f87be7c8c525 100644
--- a/content/common/gpu/media_messages.h
+++ b/content/common/gpu/media/media_messages.h
@@ -6,6 +6,7 @@
 // for a much smaller-than-usual include guard section.
 
 #include "content/common/content_export.h"
+#include "content/common/gpu/media/create_video_encoder_params.h"
 #include "gpu/config/gpu_info.h"
 #include "ipc/ipc_message_macros.h"
 #include "ipc/param_traits_macros.h"
@@ -101,24 +102,37 @@ IPC_STRUCT_TRAITS_BEGIN(media::SubsampleEntry)
   IPC_STRUCT_TRAITS_MEMBER(cypher_bytes)
 IPC_STRUCT_TRAITS_END()
 
+IPC_STRUCT_TRAITS_BEGIN(content::CreateVideoEncoderParams)
+  IPC_STRUCT_TRAITS_MEMBER(command_buffer_route_id)
+  IPC_STRUCT_TRAITS_MEMBER(input_format)
+  IPC_STRUCT_TRAITS_MEMBER(input_visible_size)
+  IPC_STRUCT_TRAITS_MEMBER(output_profile)
+  IPC_STRUCT_TRAITS_MEMBER(initial_bitrate)
+  IPC_STRUCT_TRAITS_MEMBER(encoder_route_id)
+IPC_STRUCT_TRAITS_END()
+
+// Create and initialize a hardware jpeg decoder using the specified route_id.
+// Created decoders should be freed with AcceleratedJpegDecoderMsg_Destroy when
+// no longer needed.
+IPC_SYNC_MESSAGE_CONTROL1_1(GpuChannelMsg_CreateJpegDecoder,
+                            int32_t /* route_id */,
+                            bool /* succeeded */)
+
 // Create and initialize a hardware video decoder using the specified route_id.
 // Created decoders should be freed with AcceleratedVideoDecoderMsg_Destroy when
 // no longer needed.
-IPC_SYNC_MESSAGE_ROUTED2_1(GpuCommandBufferMsg_CreateVideoDecoder,
-                           media::VideoDecodeAccelerator::Config, /* config */
-                           int32_t,                               /* route_id */
-                           bool /* succeeded */)
+IPC_SYNC_MESSAGE_CONTROL3_1(GpuCommandBufferMsg_CreateVideoDecoder,

[dcheng, 2016/03/04 02:00:11]

Btw, are these renderer to GPU messages? Or something else? Can we document that
here?

(It's hard for me to tell, since so much of the gpu bits live in content/common)

[Fady Samuel, 2016/03/04 02:30:00]

Good point! I moved them to their appropriate sections in the file.

+                            int32_t /* command_buffer_route_id */,
+                            media::VideoDecodeAccelerator::Config /* config */,
+                            int32_t /* decoder_route_id */,
+                            bool /* succeeded */)
 
 // Create and initialize a hardware video encoder using the specified route_id.
 // Created encoders should be freed with AcceleratedVideoEncoderMsg_Destroy when
 // no longer needed.
-IPC_SYNC_MESSAGE_ROUTED5_1(GpuCommandBufferMsg_CreateVideoEncoder,
-                           media::VideoPixelFormat /* input_format */,
-                           gfx::Size /* input_visible_size */,
-                           media::VideoCodecProfile /* output_profile */,
-                           uint32_t /* initial_bitrate */,
-                           int32_t, /* route_id */
-                           bool /* succeeded */)
+IPC_SYNC_MESSAGE_CONTROL1_1(GpuCommandBufferMsg_CreateVideoEncoder,
+                            content::CreateVideoEncoderParams,
+                            bool /* succeeded */)
 
 //------------------------------------------------------------------------------
 // Accelerated Video Decoder Messages