BrowserMediaPlayerManager manages MediaKeys objects.

media/base/android/media_source_player.cc

Index: media/base/android/media_source_player.cc
diff --git a/media/base/android/media_source_player.cc b/media/base/android/media_source_player.cc
index c4998e8174e6eef78d442af9cb8f3030c20fc934..2911ea58939c1b9a2c49800d49202e457d94f66f 100644
--- a/media/base/android/media_source_player.cc
+++ b/media/base/android/media_source_player.cc
@@ -374,7 +374,7 @@ void MediaSourcePlayer::OnMediaCryptoReady() {
     StartInternal();
 }
 
-void MediaSourcePlayer::SetDrmBridge(MediaDrmBridge* drm_bridge) {
+void MediaSourcePlayer::SetCdm(MediaKeys* cdm) {
   // Currently we don't support DRM change during the middle of playback, even
   // if the player is paused.
   // TODO(qinmin): support DRM change after playback has started.
@@ -384,7 +384,8 @@ void MediaSourcePlayer::SetDrmBridge(MediaDrmBridge* drm_bridge) {
             << "This is not well supported!";
   }
 
-  drm_bridge_ = drm_bridge;
+  // Only MediaDrmBridge will be set on MediaSourcePlayer.

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

This seems strange. Why did we go to all that trouble to use MediaKeys then cast
back?

Should we have kept GetMediaBridge() in BrowserMediaPlayerManager?

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

The purpose of this CL is that BrowserMediaPlayerManager can manage any CDMs,
e.g. a ChromeCast CDM. So I'd keep any platform specific class out of the
manager.

+  drm_bridge_ = static_cast<MediaDrmBridge*>(cdm);
 
   if (drm_bridge_->GetMediaCrypto().is_null()) {
     drm_bridge_->SetMediaCryptoReadyCB(base::Bind(