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

Unified Diff: media/blink/webmediaplayer_delegate.h

Issue 1570043002: Implement MediaSession on top of the WebMediaPlayerDelegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@media_session
Patch Set: Reorder. Created 4 years, 11 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
« no previous file with comments | « media/blink/webmediaplayer_cast_android.cc ('k') | media/blink/webmediaplayer_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/blink/webmediaplayer_delegate.h
diff --git a/media/blink/webmediaplayer_delegate.h b/media/blink/webmediaplayer_delegate.h
index 6cd96200814026c830bc9172194d8d044a59629d..812251fedf830e35e7e224be6c039c7bbf5908d1 100644
--- a/media/blink/webmediaplayer_delegate.h
+++ b/media/blink/webmediaplayer_delegate.h
@@ -18,25 +18,35 @@ class WebMediaPlayerDelegate {
public:
virtual void OnHidden() = 0;
virtual void OnShown() = 0;
+ virtual void OnPlay() = 0;
+ virtual void OnPause() = 0;
+
+ // Playout volume should be set to current_volume * multiplier. The range
+ // is [0, 1] and is typically 1.
+ virtual void OnVolumeMultiplierUpdate(double multiplier) = 0;
};
WebMediaPlayerDelegate() {}
+ // Subscribe or unsubscribe from observer callbacks respectively. A client
+ // must use the delegate id returned by AddObserver() for all other calls.
+ virtual int AddObserver(Observer* observer) = 0;
+ virtual void RemoveObserver(int delegate_id) = 0;
+
// The specified player started playing media.
- virtual void DidPlay(blink::WebMediaPlayer* player) = 0;
+ virtual void DidPlay(int delegate_id,
+ bool has_video,
+ bool has_audio,
+ bool is_remote,
+ base::TimeDelta duration) = 0;
// The specified player stopped playing media.
- virtual void DidPause(blink::WebMediaPlayer* player) = 0;
-
- // The specified player was destroyed. Do not call any methods on it.
- // (RemoveObserver() is still necessary if the player is also an observer.)
- virtual void PlayerGone(blink::WebMediaPlayer* player) = 0;
-
- // Subscribe to observer callbacks.
- virtual void AddObserver(Observer* observer) = 0;
+ virtual void DidPause(int delegate_id, bool reached_end_of_stream) = 0;
- // Unsubscribe from observer callbacks.
- virtual void RemoveObserver(Observer* observer) = 0;
+ // The specified player was destroyed or suspended. This may be called
+ // multiple times in row. Note: Clients must still call RemoveObserver() to
+ // unsubscribe from callbacks.
+ virtual void PlayerGone(int delegate_id) = 0;
// Returns whether the render frame is currently hidden.
virtual bool IsHidden() = 0;
« no previous file with comments | « media/blink/webmediaplayer_cast_android.cc ('k') | media/blink/webmediaplayer_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698