Add CdmService to keep track of CDMs

content/public/common/cdm_info.h

+// 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.
+
+#ifndef CONTENT_PUBLIC_COMMON_CDM_INFO_H_
+#define CONTENT_PUBLIC_COMMON_CDM_INFO_H_
+
+#include <string>
+#include <vector>
+
+#include "base/files/file_path.h"
+#include "base/version.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+// Represents a Content Decryption Module implementation and its capabilities.
+struct CONTENT_EXPORT CdmInfo {
+  CdmInfo(const std::string& name,
+          const base::Version& version,
+          const base::FilePath& path,
+          const std::vector<std::string>& supported_codecs);
+  ~CdmInfo();
+
+  // Identifier for the CDM (e.g. Widevine).

[xhwang, 2016/03/26 07:46:34]

nit: s/Identifier/Name?

[jrummell, 2016/03/29 23:48:45]

Done.

+  std::string name;

[xhwang, 2016/03/26 07:46:34]

Not for this CL. I wonder whether we should use enum for the CDM name/type. BTW,
in the FileIO CL we are using "type". We'd better be consistent.

[jrummell, 2016/03/29 23:48:45]

Could be done, but then Cast would need to change it to add any additional CDMs.
This way it's just a string.

[xhwang, 2016/03/30 05:11:18]

Can consolidate on "name" vs "type" (as in the file io cl)?

[jrummell, 2016/04/01 21:15:53]

Switched to "type".

+
+  // Version of the CDM. May be empty if the version is not known.
+  base::Version version;
+
+  // Path to the library implementing the CDM. May be empty if the
+  // CDM is always available (e.g. Widevine on Android).

[xhwang, 2016/03/26 07:46:34]

On Linux our WV CDM is also always available... It seems the fundamental
difference is the CDM is a separate library.

[jrummell, 2016/03/29 23:48:45]

Done.

+  base::FilePath path;
+
+  // List of codecs supported by the CDM (e.g. webm).

[xhwang, 2016/03/26 07:46:34]

nit: "webm" is not a codec. You can use "vp8" as an example.

[jrummell, 2016/03/29 23:48:45]

Done.

+  std::vector<std::string> supported_codecs;

[xhwang, 2016/03/26 07:46:34]

We should probably use VideoCodec and AudioCodec here. Since this will affect
existing code (e.g. component installer, IPC), we can do it later. Could you
please add a TODO?

https://code.google.com/p/chromium/codesearch#chromium/src/media/base/video_c...

+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_COMMON_CDM_INFO_H_