Chromium Code Reviews Chromium Code Reviews
Issue 2243473002:
Adding overflow menu to media player (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-fullscreen-button.html |
| diff --git a/third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-fullscreen-button.html b/third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-fullscreen-button.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..c14657ec17ce0285fac89eafbebf2a433e332228 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-fullscreen-button.html |
| @@ -0,0 +1,38 @@ |
| +<!DOCTYPE html> |
| +<title>Clicking on the overflow fullscreen button opens the video in fullscreen.</title> |
| +<script src="../resources/testharness.js"></script> |
| +<script src="../resources/testharnessreport.js"></script> |
| +<script src="media-controls.js"></script> |
| +<script src="media-file.js"></script> |
| +<script src="overflow-menu.js"></script> |
| + |
| +<!--Padding ensures the overflow menu is visible for the tests. --> |
| +<body style="padding-top: 200px; padding-left: 100px"> |
| +<video controls></video> |
| +<script> |
| +async_test(function(t) { |
| + // Set up video |
| + var video = document.querySelector("video"); |
| + video.src = findMediaFile("video", "content/test"); |
| + video.setAttribute("width", "60"); |
| + window.addEventListener("load", t.step_func(function() { |
| + var overflowList = getOverflowList(video); |
| + var overflowMenu = getOverflowMenuButton(video); |
| + |
| + // Click on the overflow menu button |
| + var coords = elementCoordinates(overflowMenu); |
| + clickAtCoordinates(coords[0], coords[1]); |
| + |
| + // Clicking on the fullscreen button should open up the video in fullscreen |
| + var coords = elementCoordinates(overflowList.children[OverflowMenuButtons.FULLSCREEN]); |
| + clickAtCoordinates(coords[0], coords[1]); |
| + |
| + video.onwebkitfullscreenchange = t.step_func(function() { |
|
mlamouri (slow - plz ping)
2016/09/07 09:07:54
This is still using the prefixed version. Can you
kdsilva
2016/09/07 11:01:09
Done.
|
| + // Overflow menu button should not be visible in fullscreen |
| + assert_equals(getComputedStyle(overflowMenu).display, "none"); |
| + t.done(); |
| + }); |
| + })); |
| +}); |
| +</script> |
| +</body> |
