From 8fc0f5be02eee83d34830fe0605b30ab36a06d18 Mon Sep 17 00:00:00 2001 From: "xhwang@chromium.org" Date: Sat, 7 Jun 2014 20:28:18 +0000 Subject: Separate BrowserCdmManager from BrowserMediaPlayerManager. BUG=315312 TEST=Test pages still work. Review URL: https://codereview.chromium.org/315733003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275695 0039d316-1c4b-4281-b951-d872f2087c98 --- media/base/android/media_player_manager.h | 39 ---------------------- media/base/android/media_source_player_unittest.cc | 14 -------- 2 files changed, 53 deletions(-) (limited to 'media') diff --git a/media/base/android/media_player_manager.h b/media/base/android/media_player_manager.h index d7ee791..8975552f 100644 --- a/media/base/android/media_player_manager.h +++ b/media/base/android/media_player_manager.h @@ -5,21 +5,13 @@ #ifndef MEDIA_BASE_ANDROID_MEDIA_PLAYER_MANAGER_H_ #define MEDIA_BASE_ANDROID_MEDIA_PLAYER_MANAGER_H_ -#include -#include - #include "base/basictypes.h" #include "base/time/time.h" #include "media/base/android/demuxer_stream_player_params.h" #include "media/base/media_export.h" -#include "media/base/media_keys.h" -#include "url/gurl.h" - -class GURL; namespace media { -class BrowserCdm; class MediaPlayerAndroid; class MediaResourceGetter; @@ -74,39 +66,8 @@ class MEDIA_EXPORT MediaPlayerManager { // Release all the players managed by this object. virtual void DestroyAllMediaPlayers() = 0; - // Get the CDM for the given CDM ID. - virtual BrowserCdm* GetCdm(int cdm_id) = 0; - // Called by the player to get a hardware protected surface. virtual void RequestFullScreen(int player_id) = 0; - - // The following five methods are related to EME. - // TODO(xhwang): These methods needs to be decoupled from MediaPlayerManager - // to support the W3C Working Draft version of the EME spec. - // http://crbug.com/315312 - - // Called when CDM creates a session. - virtual void OnSessionCreated(int cdm_id, - uint32 session_id, - const std::string& web_session_id) = 0; - - // Called when CDM wants to send a Message event. - virtual void OnSessionMessage(int cdm_id, - uint32 session_id, - const std::vector& message, - const GURL& destination_url) = 0; - - // Called when CDM wants to send a Ready event. - virtual void OnSessionReady(int cdm_id, uint32 session_id) = 0; - - // Called when CDM wants to send a Closed event. - virtual void OnSessionClosed(int cdm_id, uint32 session_id) = 0; - - // Called when CDM wants to send an Error event. - virtual void OnSessionError(int cdm_id, - uint32 session_id, - MediaKeys::KeyError error_code, - uint32 system_code) = 0; }; } // namespace media diff --git a/media/base/android/media_source_player_unittest.cc b/media/base/android/media_source_player_unittest.cc index 0d6c514..4889851 100644 --- a/media/base/android/media_source_player_unittest.cc +++ b/media/base/android/media_source_player_unittest.cc @@ -74,21 +74,7 @@ class MockMediaPlayerManager : public MediaPlayerManager { virtual MediaPlayerAndroid* GetFullscreenPlayer() OVERRIDE { return NULL; } virtual MediaPlayerAndroid* GetPlayer(int player_id) OVERRIDE { return NULL; } virtual void DestroyAllMediaPlayers() OVERRIDE {} - virtual BrowserCdm* GetCdm(int cdm_id) OVERRIDE { return NULL; } virtual void RequestFullScreen(int player_id) OVERRIDE {} - virtual void OnSessionCreated(int cdm_id, - uint32 session_id, - const std::string& web_session_id) OVERRIDE {} - virtual void OnSessionMessage(int cdm_id, - uint32 session_id, - const std::vector& message, - const GURL& destination_url) OVERRIDE {} - virtual void OnSessionReady(int cdm_id, uint32 session_id) OVERRIDE {} - virtual void OnSessionClosed(int cdm_id, uint32 session_id) OVERRIDE {} - virtual void OnSessionError(int cdm_id, - uint32 session_id, - MediaKeys::KeyError error_code, - uint32 system_code) OVERRIDE {} bool playback_completed() const { return playback_completed_; -- cgit v1.1