Chromium Code Reviews Chromium Code Reviews
Issue 336303002:
Fix HTMLMediaElement to always use the 'effective playback rate' (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/media/media-controller-effective-playback-rate.html |
| diff --git a/LayoutTests/media/media-controller-effective-playback-rate.html b/LayoutTests/media/media-controller-effective-playback-rate.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..b6d7936f329e4f621be568a307e4fa6d7e8d2e06 |
| --- /dev/null |
| +++ b/LayoutTests/media/media-controller-effective-playback-rate.html |
| @@ -0,0 +1,49 @@ |
| +<!DOCTYPE html> |
| +<html> |
| + <head> |
| + <title>Test effective playback rate with a MediaController.</title> |
| + <script src="media-file.js"></script> |
| + <script src="video-test.js"></script> |
| + <script> |
| + var start = function() |
| + { |
| + findMediaElement(); |
| + video.src = findMediaFile("video", "content/test"); |
| + |
| + // Set the rate on the media element so playback won't move forward. |
| + video.playbackRate = 0; |
| + |
| + waitForEvent("canplay",canplay); |
| + }; |
| + |
| + var canplay = function() |
| + { |
| + // Assign a mediaGroup so a controller is created. |
| + video.mediaGroup = "group"; |
|
philipj_slow
2014/06/17 12:10:54
Just video.controller = new MediaController() also
acolwell GONE FROM CHROMIUM
2014/06/17 16:55:18
Done.
|
| + |
| + // Verify the controller playback rate is 1. This |
| + // means the "effective playback rate" is 1 even though |
| + // the HTMLMediaElement's playbackRate attribute is 0. |
| + testExpected("video.controller.playbackRate", 1); |
| + testExpected("video.playbackRate", 0); |
| + |
| + // Initiate playback and verify that time moves forward. |
| + run("video.play()"); |
| + waitForEventOnce("play", play); |
| + }; |
| + |
| + var play = function() |
| + { |
| + waitForEventOnce("timeupdate", timeupdate); |
| + }; |
| + |
| + var timeupdate = function() { |
| + testExpected("video.currentTime", 0, "!="); |
| + endTest(); |
| + }; |
| + </script> |
| + </head> |
| + <body onload="start()"> |
| + <video controls></video> |
|
philipj_slow
2014/06/17 12:10:54
The controls attribute doesn't seem needed for the
acolwell GONE FROM CHROMIUM
2014/06/17 16:55:18
Done.
|
| + </body> |
| +</html> |
