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

Unified Diff: LayoutTests/media/video-controls-focus-movement-on-hide.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-focus-movement-on-hide.html
diff --git a/LayoutTests/media/video-controls-focus-movement-on-hide.html b/LayoutTests/media/video-controls-focus-movement-on-hide.html
index bd378a350f18923dac64305f28cb49e52fb14563..38d37d4f42894d3dd0648a4ca30d95ce87be0d1b 100644
--- a/LayoutTests/media/video-controls-focus-movement-on-hide.html
+++ b/LayoutTests/media/video-controls-focus-movement-on-hide.html
@@ -20,20 +20,17 @@ video.addEventListener("canplay", function()
playButton.click();
testExpected("document.activeElement", video);
-});
-video.addEventListener("timeupdate", function(event)
-{
- var video = event.target;
- if (video.currentTime < 4)
- return;
- controls = mediaControlsButton(video, "panel");
- testExpected("getComputedStyle(controls).opacity", 0);
+ runAfterControlsHidden(function()
+ {
+ controls = mediaControlsButton(video, "panel");
+ testExpected("getComputedStyle(controls).opacity", 0);
- testExpected("document.activeElement", document.body);
+ testExpected("document.activeElement", document.body);
- consoleWrite("");
- endTest();
+ consoleWrite("");
+ endTest();
+ }, video);
});
</script>
« no previous file with comments | « LayoutTests/media/media-controls.js ('k') | LayoutTests/media/video-controls-hide-on-move-outside-controls.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698