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

Side by Side Diff: third_party/WebKit/LayoutTests/media/mediasession/mojo/media-control-set-handler-notifies-service.html

Issue 2589893002: [Blink>MediaSession] Use setActionCallback() instead of event listeners for media control actions (Closed)
Patch Set: more 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 unified diff | Download patch
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <title>Test that setting MediaSession event handler should notify the service</t itle> 2 <title>Test that setting MediaSession event handler should notify the service</t itle>
3 <script src="../../../resources/testharness.js"></script> 3 <script src="../../../resources/testharness.js"></script>
4 <script src="../../../resources/testharnessreport.js"></script> 4 <script src="../../../resources/testharnessreport.js"></script>
5 <script src="../../../resources/mojo-helpers.js"></script> 5 <script src="../../../resources/mojo-helpers.js"></script>
6 <script src="resources/mediasessionservice-mock.js"></script> 6 <script src="resources/mediasessionservice-mock.js"></script>
7 <script src="resources/utils.js"></script> 7 <script src="resources/utils.js"></script>
8 <script> 8 <script>
9 9
10 var expectations; 10 var expectations;
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 m.setEnableDisableActionCallback(t.step_func(function(action, isEnabled) { 49 m.setEnableDisableActionCallback(t.step_func(function(action, isEnabled) {
50 var expectedAction = getExpectations()[nextExpectation][0]; 50 var expectedAction = getExpectations()[nextExpectation][0];
51 var expectedIsEnabled = getExpectations()[nextExpectation][1]; 51 var expectedIsEnabled = getExpectations()[nextExpectation][1];
52 assert_equals(expectedAction, action); 52 assert_equals(expectedAction, action);
53 assert_equals(expectedIsEnabled, isEnabled); 53 assert_equals(expectedIsEnabled, isEnabled);
54 if (++nextExpectation >= getExpectations().length) 54 if (++nextExpectation >= getExpectations().length)
55 t.done(); 55 t.done();
56 })); 56 }));
57 57
58 // Setting handlers 58 // Setting handlers
59 window.navigator.mediaSession.onresize = _ => {}; // Unknown event. 59 // window.navigator.mediaSession.setActionCallback("resize", _ => {}); // U nknown event.
60 window.navigator.mediaSession.onplay = _ => {}; 60 window.navigator.mediaSession.setActionCallback("play", _ => {});
61 window.navigator.mediaSession.onpause = _ => {}; 61 window.navigator.mediaSession.setActionCallback("pause", _ => {});
62 window.navigator.mediaSession.onprevioustrack = _ => {}; 62 window.navigator.mediaSession.setActionCallback("previoustrack", _ => {});
63 window.navigator.mediaSession.onnexttrack = _ => {}; 63 window.navigator.mediaSession.setActionCallback("nexttrack", _ => {});
64 window.navigator.mediaSession.onseekbackward = _ => {}; 64 window.navigator.mediaSession.setActionCallback("seekbackward", _ => {});
65 window.navigator.mediaSession.onseekforward = _ => {}; 65 window.navigator.mediaSession.setActionCallback("seekforward", _ => {});
66 66
67 // Resetting handlers 67 // Resetting handlers
68 window.navigator.mediaSession.onresize = _ => {}; // Unknown event. 68 // window.navigator.mediaSession.setActionCallback("resize", _ => {}); // U nknown event.
69 window.navigator.mediaSession.onplay = _ => {}; 69 window.navigator.mediaSession.setActionCallback("play", _ => {});
70 window.navigator.mediaSession.onpause = _ => {}; 70 window.navigator.mediaSession.setActionCallback("pause", _ => {});
71 window.navigator.mediaSession.onprevioustrack = _ => {}; 71 window.navigator.mediaSession.setActionCallback("previoustrack", _ => {});
72 window.navigator.mediaSession.onnexttrack = _ => {}; 72 window.navigator.mediaSession.setActionCallback("nexttrack", _ => {});
73 window.navigator.mediaSession.onseekbackward = _ => {}; 73 window.navigator.mediaSession.setActionCallback("seekbackward", _ => {});
74 window.navigator.mediaSession.onseekforward = _ => {}; 74 window.navigator.mediaSession.setActionCallback("seekforward", _ => {});
75 75
76 // Unsetting handlers 76 // Unsetting handlers
77 window.navigator.mediaSession.onresize = null; // Unknown event. 77 // window.navigator.mediaSession.setActionCallback("resize", null); // Unkn own event.
78 window.navigator.mediaSession.onplay = null; 78 window.navigator.mediaSession.setActionCallback("play", null);
79 window.navigator.mediaSession.onpause = null; 79 window.navigator.mediaSession.setActionCallback("pause", null);
80 window.navigator.mediaSession.onprevioustrack = null; 80 window.navigator.mediaSession.setActionCallback("previoustrack", null);
81 window.navigator.mediaSession.onnexttrack = null; 81 window.navigator.mediaSession.setActionCallback("nexttrack", null);
82 window.navigator.mediaSession.onseekbackward = null; 82 window.navigator.mediaSession.setActionCallback("seekbackward", null);
83 window.navigator.mediaSession.onseekforward = null; 83 window.navigator.mediaSession.setActionCallback("seekforward", null);
84 }); 84 });
85 }, "test that setting event handler notifies the mojo service"); 85 }, "test that setting event handler notifies the mojo service");
86 86
87 </script> 87 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698