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

Unified Diff: LayoutTests/media/video-controls-hide-on-move-outside-controls.html

Issue 302603003: Cleanup tests related to controls hiding/fading in/out (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Tweaks. Created 6 years, 7 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: LayoutTests/media/video-controls-hide-on-move-outside-controls.html
diff --git a/LayoutTests/media/video-controls-hide-on-move-outside-controls.html b/LayoutTests/media/video-controls-hide-on-move-outside-controls.html
index ddea8decb1701046a0f9fc488ae7e5e62bc5d7ef..bbd27246f8be2dbc2a2796618cdb0ae7283f0646 100644
--- a/LayoutTests/media/video-controls-hide-on-move-outside-controls.html
+++ b/LayoutTests/media/video-controls-hide-on-move-outside-controls.html
@@ -15,16 +15,6 @@ var controls;
function runTest()
{
video = document.getElementById("no-video-media");
- video.addEventListener("timeupdate", function()
- {
- if (video.currentTime < 4)
- return;
- controls = mediaControlsButton(video, "panel");
- testExpected("getComputedStyle(controls).opacity", 0);
-
- consoleWrite("");
- endTest();
- });
testExpected("video.paused", true);
if (!window.testRunner)
@@ -39,6 +29,15 @@ function runTest()
eventSender.mouseUp();
testExpected("video.paused", false);
+ runAfterControlsHidden(function()
+ {
+ controls = mediaControlsButton(video, "panel");
+ testExpected("getComputedStyle(controls).opacity", 0);
+
+ consoleWrite("");
+ endTest();
+ }, video);
+
// Move the mouse to the upper-left corner of the video.
eventSender.mouseMoveTo(video.offsetLeft + 4, video.offsetTop + 4);
}
« no previous file with comments | « LayoutTests/media/video-controls-focus-movement-on-hide.html ('k') | LayoutTests/media/video-controls-show-on-focus.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698