BrowserMediaPlayerManager manages MediaKeys objects.

media/base/android/cdm_factory.h

+// Copyright 2014 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_ANDROID_CDM_FACTORY_H_
+#define MEDIA_BASE_ANDROID_CDM_FACTORY_H_
+
+#include <string>
+
+#include "base/memory/scoped_ptr.h"
+#include "media/base/media_export.h"
+#include "media/base/media_keys.h"
+
+namespace media {
+
+// Creates a CDM for |key_system|. Returns NULL if the CDM cannot be created.
+scoped_ptr<MediaKeys> MEDIA_EXPORT
+    CreateCdm(const std::string& key_system,

[ddorwin, 2014/05/13 00:59:38]

This name could conflict with other code in media. If this is really for
Android, maybe we should rename it CreateAndroidCdm. Otherwise, should we be
using the factory function for desktop too?

[damienv1, 2014/05/13 17:07:07]

This is a static function that is invoked to create a platform specific CDM so
CreateAndroidCdm is not suitable (the same static function will be used to
create a Chromecast CDM).

[ddorwin, 2014/05/13 17:17:37]

If it's only used for platform-based (browser process) CDMs, maybe this should
be CreateBrowserCdm() or something.

[xhwang, 2014/05/14 16:42:06]

Done.

+              const SessionCreatedCB& session_created_cb,
+              const SessionMessageCB& session_message_cb,
+              const SessionReadyCB& session_ready_cb,
+              const SessionClosedCB& session_closed_cb,
+              const SessionErrorCB& session_error_cb);
+
+}  // namespace media
+
+#endif  // MEDIA_BASE_ANDROID_CDM_FACTORY_H_

[damienv1, 2014/05/12 23:14:26]

This file could already be part of media/base/ (at the same level as MediaKeys).
Nothing is android specific here.

[xhwang, 2014/05/14 16:42:06]

Good point. Done.