LayoutTests/media/video-controls-focus-movement-on-hide.html - Issue 297783004: Implement heuristic for showing media controls during playback w/o a mouse

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: LayoutTests/media/video-controls-focus-movement-on-hide.html

Issue 297783004: Implement heuristic for showing media controls during playback w/o a mouse (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: IgnoreSelfHover -> IgnoreVideoHover; Tweak comment. 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:

OLD	NEW
(Empty)
	1 <!DOCTYPE html>

	2 <html>

	3 <title>Test focus movement when controls fade out with a button focused</title>

	4 <script src=video-test.js></script>

	5 <script src=media-controls.js></script>

	6 <p>

	7 Test that focus moves to the <body>/document when controls fade out with a bu tton focused.

	8 </p>

	9 <video controls src="content/test.ogv"></video>

	10 <script>

	11 var controls;

	12 var video = document.querySelector("video");

	13

	14 video.addEventListener("canplay", function()

	15 {

	16 testExpected("document.activeElement", document.body);

	17

	18 var playButton = mediaControlsButton(video, "play-button");

	19 playButton.focus();

	20 playButton.click();

	21

	22 testExpected("document.activeElement", video);

	23 });

	24

	25 video.addEventListener("timeupdate", function(event)

	26 {

	27 var video = event.target;

	28 if (video.currentTime < 4)
	acolwell GONE FROM CHROMIUM 2014/05/23 18:26:28 nit: I think we should define a constant in media- nit: I think we should define a constant in media-controls.js to indicate that this is enough time for the controls to auto-hide. I think that will make it easier to fix broken tests if this value ever changes.
	29 return;

	30 controls = mediaControlsButton(video, "panel");

	31 testExpected("getComputedStyle(controls).opacity", 0);

	32

	33 testExpected("document.activeElement", document.body);

	34

	35 consoleWrite("");

	36 endTest();

	37 });

	38

	39 </script>

OLD	NEW