third_party/WebKit/LayoutTests/media/video-controls-captions-on-off.html - Issue 1079323002: Support text track selection in video controls

Unified Diff: third_party/WebKit/LayoutTests/media/video-controls-captions-on-off.html

Issue 1079323002: Support text track selection in video controls (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Rebase and addressed fs' comments Created 4 years, 10 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

« no previous file with comments | « third_party/WebKit/LayoutTests/media/video-controls-captions-multiple-clicks-expected.txt ('k') | third_party/WebKit/LayoutTests/media/video-controls-captions-on-off-expected.txt » ('j') | third_party/WebKit/Source/core/css/CSSValueKeywords.in » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/media/video-controls-captions-on-off.html

diff --git a/third_party/WebKit/LayoutTests/media/video-controls-captions-multiple-clicks.html b/third_party/WebKit/LayoutTests/media/video-controls-captions-on-off.html

similarity index 78%

rename from third_party/WebKit/LayoutTests/media/video-controls-captions-multiple-clicks.html

rename to third_party/WebKit/LayoutTests/media/video-controls-captions-on-off.html

index ad4d5e5742e7295a5d20540969389451c438432e..b6a729fa0a4dbd56614d59f52ac4da32968dd1a6 100644

--- a/third_party/WebKit/LayoutTests/media/video-controls-captions-multiple-clicks.html

+++ b/third_party/WebKit/LayoutTests/media/video-controls-captions-on-off.html

@@ -2,22 +2,21 @@

<html>

<head>

- <title>Test closed caption button toggling.</title>

+ <title>Test closed caption track selection on and off.</title>

<!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956

(Please avoid writing new tests using video-test.js) -->

- var displayElement;

- var track;

var text = ["First", "Second", "Third"];

+ var displayElement;

function addTextTrack()

{

- track = video.addTextTrack('captions');

+ var track = video.addTextTrack('captions');

- for(var i = 0; i < 3; i++) {

+ for (var i = 0; i < 3; i++) {

var cue = new VTTCue(0, 120, text[i]);

track.addCue(cue);

}

@@ -27,9 +26,9 @@

{

for (var i = 0; i < 3; i++) {

try {

- displayElement = textTrackDisplayElement(video, 'display', i);

+ displayElement = textTrackDisplayElement(video, "display", i);

testExpected("displayElement.innerText", text[i]);

- } catch(e) {

+ } catch (e) {

consoleWrite(e);

}

@@ -54,18 +53,18 @@

checkCaptionsDisplay();

consoleWrite("");

- consoleWrite("** Captions track should become visible after button is clicked **");

- clickCCButton();

+ consoleWrite("** Captions track should become visible after the track is selected **");

+ selectTextTrack(video, 0);

checkCaptionsDisplay();

consoleWrite("");

- consoleWrite("** Captions should not be visible after button is clicked again **");

- clickCCButton();

+ consoleWrite("** Captions should not be visible after they're turned off through the menu **");

+ turnClosedCaptionsOff(video);

checkCaptionsDisplay();

consoleWrite("");

- consoleWrite("** Captions should become visible after button is clicked again **");

- clickCCButton();

+ consoleWrite("** Captions track should become visible after the track is selected again **");

+ selectTextTrack(video, 0);

checkCaptionsDisplay();

consoleWrite("");

@@ -82,7 +81,7 @@

</script>

</head>

- <p>Tests that multiple toggles of the closed captions button still display captions</p>

+ <p>Tests that tracks can be turned on and off through the track selection menu</p>

</body>

</html>