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

Unified Diff: third_party/WebKit/Source/modules/mediasession/MediaSession.idl

Issue 2589893002: [Blink>MediaSession] Use setActionCallback() instead of event listeners for media control actions (Closed)
Patch Set: rebased and more tests Created 4 years 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: third_party/WebKit/Source/modules/mediasession/MediaSession.idl
diff --git a/third_party/WebKit/Source/modules/mediasession/MediaSession.idl b/third_party/WebKit/Source/modules/mediasession/MediaSession.idl
index bd294bc66219ff71c5da36e1c4a47ee394748293..c3d9ddd7c74d7ae93c34524565e8312894e16dad 100644
--- a/third_party/WebKit/Source/modules/mediasession/MediaSession.idl
+++ b/third_party/WebKit/Source/modules/mediasession/MediaSession.idl
@@ -12,17 +12,24 @@ enum MediaSessionPlaybackState {
// https://wicg.github.io/mediasession/#the-mediasession-interface
+enum MediaSessionAction {
+ "play",
+ "pause",
+ "previoustrack",
+ "nexttrack",
+ "seekbackward",
+ "seekforward"
+};
+
+callback MediaSessionActionHandler = void ();
+
[
+ Custom=VisitDOMWrapper,
RuntimeEnabled=MediaSession,
] interface MediaSession : EventTarget {
attribute MediaMetadata? metadata;
-
attribute MediaSessionPlaybackState playbackState;
- attribute EventHandler onplay;
- attribute EventHandler onpause;
- attribute EventHandler onprevioustrack;
- attribute EventHandler onnexttrack;
- attribute EventHandler onseekforward;
- attribute EventHandler onseekbackward;
+ void setActionHandler(MediaSessionAction action,
+ MediaSessionActionHandler? handler);
};

Powered by Google App Engine
This is Rietveld 408576698