Struct traits used by GpuInfo

gpu/ipc/common/gpu_info.mojom

Index: gpu/ipc/common/gpu_info.mojom
diff --git a/gpu/ipc/common/gpu_info.mojom b/gpu/ipc/common/gpu_info.mojom
index 26b83872e7671082c154774fad6cea1185265ec6..e99aa0c15e29f7310d6878f559701b5c73c5547a 100644
--- a/gpu/ipc/common/gpu_info.mojom
+++ b/gpu/ipc/common/gpu_info.mojom
@@ -1,9 +1,11 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
+// Copyright 2016 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.
 
 module gpu.mojom;
 
+import "ui/gfx/geometry/mojo/geometry.mojom";
+
 // gpu::GPUInfo::GPUDevice
 struct GpuDevice {
   uint32 vendor_id;
@@ -19,3 +21,47 @@ enum CollectInfoResult {
   kCollectInfoNonFatalFailure = 2,
   kCollectInfoFatalFailure = 3
 };
+
+// This should always match gpu::VideoCodecProfile
+enum VideoCodecProfile {
+  VIDEO_CODEC_PROFILE_UNKNOWN = -1,
+  VIDEO_CODEC_PROFILE_MIN = VIDEO_CODEC_PROFILE_UNKNOWN,

[Fady Samuel, 2016/07/18 15:59:39]

Let's get rid of min and max. We don't need them in the wire format. They exist
for validation but StructTraits will take care of that for us.

[Alex Z., 2016/07/19 15:13:19]

Done.

+  H264PROFILE_BASELINE = 0,
+  H264PROFILE_MAIN,
+  H264PROFILE_EXTENDED,
+  H264PROFILE_HIGH,
+  H264PROFILE_HIGH10PROFILE,
+  H264PROFILE_HIGH422PROFILE,
+  H264PROFILE_HIGH444PREDICTIVEPROFILE,
+  H264PROFILE_SCALABLEBASELINE,
+  H264PROFILE_SCALABLEHIGH,
+  H264PROFILE_STEREOHIGH,
+  H264PROFILE_MULTIVIEWHIGH,
+  VP8PROFILE_ANY,
+  VP9PROFILE_PROFILE0,
+  VP9PROFILE_PROFILE1,
+  VP9PROFILE_PROFILE2,
+  VP9PROFILE_PROFILE3,
+  HEVCPROFILE_MAIN,
+  HEVCPROFILE_MAIN10,
+  HEVCPROFILE_MAIN_STILL_PICTURE,
+  VIDEO_CODEC_PROFILE_MAX = HEVCPROFILE_MAIN_STILL_PICTURE,
+};
+
+struct VideoDecodeAcceleratorSupportedProfile {

[Fady Samuel, 2016/07/18 15:59:39]

nit: comment with path to C++ type.

[Alex Z., 2016/07/19 15:13:19]

Done.

+  VideoCodecProfile profile;
+  gfx.mojom.Size max_resolution;
+  gfx.mojom.Size min_resolution;
+  bool encrypted_only;
+};
+
+struct VideoDecodeAcceleratorCapabilities {

[Fady Samuel, 2016/07/18 15:59:39]

nit: comment with path to C++ type.

[Alex Z., 2016/07/19 15:13:19]

Done.

+  uint32 flags;
+};
+
+struct VideoEncodeAcceleratorSupportedProfile {

[Fady Samuel, 2016/07/18 15:59:39]

nit: comment with path to C++ type.

[Alex Z., 2016/07/19 15:13:19]

Done.

+  VideoCodecProfile profile;
+  gfx.mojom.Size max_resolution;
+  uint32 max_framerate_numerator;
+  uint32 max_framerate_denominator;
+};