Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(24)

Unified Diff: media/base/android/media_source_player.h

Issue 308073004: Add PlayerTracker and BrowserCdm interfaces. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: comments addressed Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/android/media_source_player.h
diff --git a/media/base/android/media_source_player.h b/media/base/android/media_source_player.h
index 6389ec1687dade77426c59e92e504aaefeb19ddb..154cbeab4e89856f2b08a75f217cd1b5aaf1634a 100644
--- a/media/base/android/media_source_player.h
+++ b/media/base/android/media_source_player.h
@@ -61,8 +61,7 @@ class MEDIA_EXPORT MediaSourcePlayer : public MediaPlayerAndroid,
virtual bool CanSeekForward() OVERRIDE;
virtual bool CanSeekBackward() OVERRIDE;
virtual bool IsPlayerReady() OVERRIDE;
- virtual void SetCdm(MediaKeys* cdm) OVERRIDE;
- virtual void OnKeyAdded() OVERRIDE;
+ virtual void SetCdm(BrowserCdm* cdm) OVERRIDE;
virtual bool IsSurfaceInUse() const OVERRIDE;
// DemuxerAndroidClient implementation.
@@ -171,6 +170,12 @@ class MEDIA_EXPORT MediaSourcePlayer : public MediaPlayerAndroid,
// Sets the demuxer configs for audio or video stream.
void SetDemuxerConfigs(const DemuxerConfigs& configs, bool is_audio);
+ // Called when new decryption key becomes available.
+ void OnKeyAdded();
+
+ // Called when the CDM is destroyed.
ddorwin 2014/06/02 20:20:29 update comment.
xhwang 2014/06/02 21:41:22 Done.
+ void OnCdmUnset();
+
// Test-only method to setup hook for the completion of the next decode cycle.
// This callback state is cleared when it is next run.
// Prevent usage creep by only calling this from the
@@ -273,6 +278,7 @@ class MEDIA_EXPORT MediaSourcePlayer : public MediaPlayerAndroid,
base::CancelableClosure decoder_starvation_callback_;
MediaDrmBridge* drm_bridge_;
+ int cdm_registration_id_;
// No decryption key available to decrypt the encrypted buffer. In this case,
// the player should pause. When a new key is added (OnKeyAdded()), we should
@@ -292,6 +298,7 @@ class MEDIA_EXPORT MediaSourcePlayer : public MediaPlayerAndroid,
// Weak pointer passed to media decoder jobs for callbacks.
// NOTE: Weak pointers must be invalidated before all other member variables.
base::WeakPtrFactory<MediaSourcePlayer> weak_factory_;
+ base::WeakPtr<MediaSourcePlayer> weak_this_;
DISALLOW_COPY_AND_ASSIGN(MediaSourcePlayer);
};

Powered by Google App Engine
This is Rietveld 408576698