New media/base/decode_capabilities.h|cc

media/base/decode_capabilities.h

Index: media/base/decode_capabilities.h
diff --git a/media/base/decode_capabilities.h b/media/base/decode_capabilities.h
new file mode 100644
index 0000000000000000000000000000000000000000..f9f144f56793a21dc7c091d0b34df26ae60d5b04
--- /dev/null
+++ b/media/base/decode_capabilities.h
@@ -0,0 +1,28 @@
+// Copyright 2017 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_BASE_DECODE_CAPABILITIES_
+#define MEDIA_BASE_DECODE_CAPABILITIES_
+
+#include "media/base/media_export.h"
+#include "media/base/video_codecs.h"
+#include "media/base/video_color_space.h"
+
+namespace media {
+
+// APIs to media's decoder capabilities. Embedders may customize decoder
+// capabilities via MediaClient. See usage in mime_util_internal.cc.
+
+struct MEDIA_EXPORT VideoConfig {

[hubbe, 2017/03/23 18:14:30]

Might be nice to update VideoDecoderConfig to use one of these later.

[chcunningham, 2017/03/23 19:20:53]

I actually tried to add level to VideoDecoderConfig, but it was a pretty ugly
and massive change. Most users of VideoDecoderConfig don't care about level and
its constructor already has over 10 parameters. Dale recommended a new struct
like you see here. Down the road  MediaCapabilities queries will continue to
grow to contain things not needed/wanted in VideoDecoderConfig.

[hubbe, 2017/03/23 19:33:00]

Acknowledged.

+  VideoCodec codec;
+  VideoCodecProfile profile;
+  int level;
+  VideoColorSpace color_space;
+};
+
+MEDIA_EXPORT bool IsSupportedVideoConfig(const VideoConfig& config);
+
+}  // namespace media
+
+#endif  // MEDIA_BASE_DECODE_CAPABILITIES_