Add CdmService to keep track of CDMs

content/public/common/key_system_registration.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_KEY_SYSTEM_REGISTRATION_H_
+#define CONTENT_PUBLIC_COMMON_KEY_SYSTEM_REGISTRATION_H_
+
+#include <string>
+
+#include "base/files/file_path.h"
+#include "base/strings/string16.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+// Collects information about Key Systems.
+struct CONTENT_EXPORT KeySystemRegistration {
+  // Name of the Key System.
+  std::string key_system_;

[xhwang, 2016/01/29 07:43:06]

No need to use trailing "_" for struct members.

[jrummell, 2016/03/04 01:19:41]

Done.

+
+  // Path to the library implementing the Key System. May be empty() if the
+  // Key System is already included as part of the system.
+  base::FilePath path_;

[xhwang, 2016/01/29 07:43:06]

Can we use a more specific name? Key System doesn't really have a path. It's the
underlying cdm::CDM that has a path. How about something like cdm_path, or
cdm_library_path?

[jrummell, 2016/03/04 01:19:41]

Done.

+
+  // List of codecs supported by the Key System.
+  base::string16 codecs_;

[xhwang, 2016/01/29 07:43:06]

Can we use std::vector<base::string16> for the "list" of codecs?

[jrummell, 2016/03/04 01:19:41]

Done.

+
+  KeySystemRegistration(const std::string& key_system,
+                        const base::FilePath& path,
+                        const base::string16& codecs)
+      : key_system_(key_system), path_(path), codecs_(codecs) {}
+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_COMMON_KEY_SYSTEM_REGISTRATION_H_