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

Unified Diff: third_party/WebKit/LayoutTests/media/mediasession/mojo/media-control-action-reaches-client.html

Issue 2589893002: [Blink>MediaSession] Use setActionCallback() instead of event listeners for media control actions (Closed)
Patch Set: fixed tests Created 3 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
Index: third_party/WebKit/LayoutTests/media/mediasession/mojo/media-control-action-reaches-client.html
diff --git a/third_party/WebKit/LayoutTests/media/mediasession/mojo/media-control-action-reaches-client.html b/third_party/WebKit/LayoutTests/media/mediasession/mojo/media-control-action-reaches-client.html
index 7e32f68bc9d1da19c1547f9bc5116bb5ce790761..ab4a42f7383bcadbe0d48ad0c0b85414f4e76cdf 100644
--- a/third_party/WebKit/LayoutTests/media/mediasession/mojo/media-control-action-reaches-client.html
+++ b/third_party/WebKit/LayoutTests/media/mediasession/mojo/media-control-action-reaches-client.html
@@ -20,20 +20,26 @@ var expectations = [
var nextExpectation = 0;
-function checkExpectation(t, event) {
+function checkExpectation(t, action) {
var expectedEvent = expectations[nextExpectation];
- assert_equals(expectedEvent, event.type);
+ assert_equals(expectedEvent, action);
if (++nextExpectation >= expectations.length)
t.done();
}
function runTests(t) {
- window.navigator.mediaSession.onplay = t.step_func(checkExpectation.bind(null, t));
- window.navigator.mediaSession.onpause = t.step_func(checkExpectation.bind(null, t));
- window.navigator.mediaSession.onprevioustrack = t.step_func(checkExpectation.bind(null, t));
- window.navigator.mediaSession.onnexttrack = t.step_func(checkExpectation.bind(null, t));
- window.navigator.mediaSession.onseekforward = t.step_func(checkExpectation.bind(null, t));
- window.navigator.mediaSession.onseekbackward = t.step_func(checkExpectation.bind(null, t));
+ window.navigator.mediaSession.setActionHandler(
+ "play", t.step_func(checkExpectation.bind(null, t, "play")));
+ window.navigator.mediaSession.setActionHandler(
+ "pause", t.step_func(checkExpectation.bind(null, t, "pause")));
+ window.navigator.mediaSession.setActionHandler(
+ "previoustrack", t.step_func(checkExpectation.bind(null, t, "previoustrack")));
+ window.navigator.mediaSession.setActionHandler(
+ "nexttrack", t.step_func(checkExpectation.bind(null, t, "nexttrack")));
+ window.navigator.mediaSession.setActionHandler(
+ "seekbackward", t.step_func(checkExpectation.bind(null, t, "seekbackward")));
+ window.navigator.mediaSession.setActionHandler(
+ "seekforward", t.step_func(checkExpectation.bind(null, t, "seekforward")));
mock.getClient().didReceiveAction(MediaSessionAction.PLAY);
mock.getClient().didReceiveAction(MediaSessionAction.PAUSE);

Powered by Google App Engine
This is Rietveld 408576698